-
Notifications
You must be signed in to change notification settings - Fork 10
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: clean up README examples #23
docs: clean up README examples #23
Conversation
- remove option overwrites - remove array for plain values - flags for flags, values for values
…thout-repeated-options
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.
Did you consider generating these docs?
It seems easy to get it out of sync
Not yet. The docs got written first, and guided the initial implementation. I am proposing changes to README, then will update code to match. 😄 |
(Deleted my comment with code which included multiple changes not all related to this PR.) |
values
? #19)values
? #19)If this is acceptable, happy to update the code to behave accordingly.