Merged
Conversation
I Cannot be the only person that thought "why is there a blank installation page". The other pages are really good too!
|
Thanks for opening this pull request! Please check out our contributing guidelines! ⛰️ |
Contributor
Author
|
While I could change https://github.com/orhun/git-cliff/actions/runs/16552784738/job/46810035103; I don't think it really belongs in this patch set. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1226 +/- ##
=======================================
Coverage 41.59% 41.59%
=======================================
Files 22 22
Lines 1919 1919
=======================================
Hits 798 798
Misses 1121 1121
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Congrats on merging your first pull request! ⛰️ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Adds some blurb to the main installation page, as it is the default navigated page from the docs if I click installation.
Motivation and Context
I Cannot be the only person that thought "why is there a blank installation page". The other pages are really good too!
I'm totally cool if this cannot be merged, but I was really expecting cargo install or install binaries to be on this page, so it caused a momentary disconnect that I had to look for where to find the information, instead of it being right there for me.
A list of links might be nice, but; for-now, this would at least allow continuous flow.
How Has This Been Tested?
I just wrote text in a docs page.
Screenshots / Logs (if applicable)
Just what the page looks like for me when I visited.
Types of Changes
Checklist: