-
Notifications
You must be signed in to change notification settings - Fork 15
Update to syn 0.12
#8
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
Comments
Hi @KodrAus, I just wanted to let you know that I'm working on a syn update by rewriting the crate, basically. First I just tried to update it to the new version incrementally, but I noticed that quite a lot has happened since this crate was written. Thus, updating wasn't that easy and would have required a lot of cleanup. So I figured I could just try to rewrite it. I'm already like... half done, approximately. Should be done in the next few days. Would you (in principle) accept a PR like that? |
Hi @LukasKalbertodt! Wow that sounds great to me! I'd be more than happy to just give you the commit bit so you can push changes through if you'd like. I'm really interested to see what an implementation using |
@KodrAus I'd prefer to create a PR and having someone to at least quickly skim over the new code. But thanks for invitation anyway. I'm sure there are more updates needed before the proc_macro system hits stable :P And if this wasn't clear: I upgraded to the newest version of I'll open a PR once I'm finished! |
So we can report errors much more better using spans to the actual code that prevents
auto_impl
from being valid.The text was updated successfully, but these errors were encountered: