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
Handle inter-expansion top-level args properly #290
Conversation
3413004
to
f66a8ea
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.
Technically a breaking change, but OK with that.
@TeXitoi What's breaking about it? |
just doing a cargo update will change the behavior of the program as the top level annotations of parent types will be executed. |
Well, I really doubt someone out there would put an attribute and expect it not to apply :) |
That's why i'm OK with that, but it can happen as someone might tests some things and do not remove top level attribute but is happy with the result. |
It can be considered as a bug. |
You don't exactly expect such careless programming yield good results :) |
Welcome to the real world! ;-) |
Fixes #151
Fixes #289
Fixes #113