-
Notifications
You must be signed in to change notification settings - Fork 7.5k
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
Tweak formatting of options tables #4730
Conversation
I just pushed an iteration I'm happy with! When the Netlify build finishes, have a look at the CLI refs for |
Don't merge, one more tweak coming. |
This looks great now! See https://deploy-preview-4730--docker-docs.netlify.com/engine/reference/commandline/build/ This shows how it looks like when you have multiple flags for specific flags. So in summary, you now have API flags, deprecation, and experimental badges at the level of the command and the level of the options. This is great! cc/ @londoncalling for how great this looks! |
This is awesome, thank you @mstanleyjones ! |
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.
❤️ awesome work! let me know when you're ready, but LGTM already
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.
A great improvement that our users will love. Thank you so much, @mstanleyjones ! Kicked off another Netlify build just in case.
FWIW; other enhancements we discussed;
|
Move the
deprecated
orexperimental
info from its own column (which is usually redundant) to the flag description. Make the word bold as well.cc/ @tiborvass who also asked for this.