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

Native proc_macro #72

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Native proc_macro #72

wants to merge 2 commits into from

Conversation

ta-vroom
Copy link

@ta-vroom ta-vroom commented Oct 4, 2021

With Rust 1.45 #[proc_macro_hack] is no longer required as #[proc_macro] has a native implementation (it also results in an annoying unresolved-macro-call error.

Consider futures-macro's implementation for supporting older Rust compilers.

@voidpumpkin
Copy link

@ZoeyR Could you take a look at this?
It is very distracting to see the proc macro error in IDE

@hoijui
Copy link

hoijui commented Mar 7, 2022

This is fixed in the new repo:
https://github.com/allan2/dotenvy

@stevenhansel
Copy link

https://github.com/allan2/dotenvy

While this repo is not maintained anymore, I don't see that the problem is fixed yet in the fork

@hoijui
Copy link

hoijui commented Jul 14, 2022

To be honest... I tried to figure out why I wrote that here, and I guess I just posted in the wrong issue (meant to post in #63). So.. you are right, sorry!
(even #63 is only fixed in a reworked PR though, not merged)

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.

4 participants