-
Notifications
You must be signed in to change notification settings - Fork 54
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
docs: restructure references #461
docs: restructure references #461
Conversation
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.
Hi @IronCore864 , some minor comments.
Am just focusing on the intended scope of this PR.
Hey @IronCore864 , some general feedback. The But it can be an enhancement. |
Thanks for the advice, I just added them in this PR since it's only adding a line and it's related to the moved content anyway. |
Co-authored-by: Ben Hoyt <[email protected]>
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.
Looks good to me, thanks. I also tweaked the pebble run
and pebble warnings
help descriptions to be wrapped to 80 columns manually -- it's a bit unfortunate these use a fixed wrapping and the args help uses an automatic wrapping, but oh well. This is good enough for now.
Some structural changes to the reference docs.
Changes:
cli-commands
.