-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
doc(cli): reorder sections and unify heading #10035
Conversation
Reviewer's Guide by SourceryThis PR reorders sections and unifies headings in the Flow diagram showing the document structure changesgraph TD
A[Original Document Structure] --> B[Reordered Document Structure]
subgraph Before
A1[Random Command Order] --> A2[Mixed Heading Levels]
A2 --> A3[### Options]
end
subgraph After
B1[Alphabetically Ordered Commands] --> B2[Consistent Heading Levels]
B2 --> B3[#### Options]
end
style A1 fill:#ffcccc
style A2 fill:#ffcccc
style B1 fill:#ccffcc
style B2 fill:#ccffcc
style B3 fill:#ccffcc
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @abn - I've reviewed your changes and they look great!
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Deploy preview for website ready! ✅ Preview Built with commit eb88d58. |
57aa388
to
284ac5c
Compare
284ac5c
to
d6eb108
Compare
d6eb108
to
eb88d58
Compare
This has been bugging me for a while now. This change attempts the following.
####
headings.Summary by Sourcery
Documentation:
####
.