-
Notifications
You must be signed in to change notification settings - Fork 86
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
Added docs on how to debug the derived marco #131
Conversation
b5e1592
to
6a46ea8
Compare
6a46ea8
to
12ab831
Compare
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.
Rather than putting argh_derive_expand, what do you think about moving it into examples/example.rs
, since we don't have a proper compilable example? cargo expand --example example
should still expand the macros.
If we go down this road, can you update https://github.com/google/argh/blob/master/.github/workflows/rust.yml
to make sure the example is automatically built?
12ab831
to
6e3c66d
Compare
It did not like very much
|
6e3c66d
to
d13271e
Compare
nvm I figured it out |
d13271e
to
7ad3218
Compare
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.
LGTM. Could you fix the cargo fmt
error though?
7ad3218
to
546cb58
Compare
Done |
26759aa
to
e2e5b31
Compare
e2e5b31
to
ad451ef
Compare
3c43479
to
f134f73
Compare
f134f73
to
a40a5ab
Compare
No description provided.