-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Would a PR to use Macros 1.1 be welcome #50
Comments
Also sorry for randomly dropping a bunch of issues on you on Christmas. |
No worries on the issues! I appreciate your diligence. :) I've given some thought to this, and at present, I'd rather keep derives as they are. My main argument against moving to macros 1.1 is that it adds another crate, and thus another line to That being said, it may be possible to reexport the derives from the core |
It would need to be a separate crate as it's a separate compilation mode. |
I'm okay with having another crate as long as users don't need to explicitly import it. What I was referring to was that it might be possible to, in the #[macro_reexport] extern crate rocket_derive; And then users would write: #[macro_use] extern crate rocket; Is this what you're referring to when you say it won't work? I haven't explored this myself. |
While most of your codegen won't be satisfied by Macros 1.1, your custom derives will be. Using libsyntax for custom derive is currently deprecated (and sounds like it's slated for removal in 1.15). Would you be open to a PR to pull the custom derives out into a separate crate which uses macros 1.1?
The text was updated successfully, but these errors were encountered: