Skip to content
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

Half-hearted update to 2018 edition #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Follpvosten
Copy link

Disclaimer: This was done as I was bored waiting at the airport, so it might not be perfect/complete yet.

Since proc_macro was stabilized recently, I thought it would be a nice idea to make this work on stable.
Well, surprise, proc_macro_diagnostics is still an unstable feature. But hey, it works on the current nightly, so that's something, I guess?

Apart from the obvious, I also applied rustfmt and fixed the one thing Clippy complained about.

@zmitchell
Copy link
Owner

I've been meaning to update this (and the article) to Rust 2018 at some point in the near future. I'll take a closer look at this when I update the article. Thanks for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants