-
Notifications
You must be signed in to change notification settings - Fork 5
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
Feat: Make cargo-witgen into lib so that other CLI apps can reuse it #26
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.
Looks good ! Just one suggestion. If it's ready to merge feel free to merge it
Also `parse_crate_as_file` now returns `File` and adds `parse_file`
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.
Maybe it needs documentation on public items
Co-authored-by: Coenen Benjamin <benjamin.coenen@hotmail.com>
Thanks for the quick review! I addressed most of your issues. I plan to add the AST types this week! |
Should I plan a release before the AST stuffs ? |
This creates a new
Witgen
struct that handles generating thewit
string and its output. This way other applications (specifically witme) can use#[clap(flatten)]
to inject it into their subcommand.