Skip to content
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(parser): copy and tweak structopt documentation for clap::Parser #3052

Closed
wants to merge 1 commit into from

Conversation

gibfahn
Copy link
Contributor

@gibfahn gibfahn commented Nov 26, 2021

Filing the PR to see whether this structure makes sense. If it does there are probably a bunch of fixups / renamings / other changes needed for this to be accurate.


Commits (oldest to newest)

8b6a94d docs(parser): copy and tweak structopt documentation for clap::Parser

Took a first pass through the docs, cleaning up obvious renames, and
noting the changes I happen to know about.

Refs: #2856


Took a first pass through the docs, cleaning up obvious renames, and
noting the changes I happen to know about.

Refs: clap-rs#2856
@epage
Copy link
Member

epage commented Dec 7, 2021

Sorry, in parallel to this,
https://github.com/clap-rs/clap/blob/master/examples/derive_ref/README.md was written, Feel free to add anything you see is needed!

@epage epage closed this Dec 16, 2021
@gibfahn gibfahn deleted the clap_parser_docs branch December 24, 2021 11:42
@gibfahn
Copy link
Contributor Author

gibfahn commented Dec 24, 2021

Ahh okay, wish I'd known before I did this!

It feels a little odd to have the documentation for the derive crate not be on docs.rs, do you think we could make this part of the docs somewhere, vs directly on GitHub?

@epage
Copy link
Member

epage commented Dec 24, 2021

That is being discussed in #3189.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants