SergioBenitez / proc-macro2-diagnostics

Diagnostics for stable and nightly proc-macros!
Apache License 2.0
10 stars 5 forks source link

Explicitly specify features needed for syn #5

Closed Techcable closed 1 year ago

Techcable commented 1 year ago

Requires switch to resolver = "2", so that features for dev-dependencies and regular dependencies are separate.

Otherwise, our dev-dependency on trybuild would transitively include the default syn features.

It appears the "printing" dependency is not directly required, it is only needed by the tests. However, I have added it because it's used for the 'Spanned' trait, which is likely needed in the future.

Also, we already have a dependency on the 'quote' crate, so the additional "printing" feature doesn't pull in much more.

Fixes issue #4

SergioBenitez commented 1 year ago

Finally got this in https://github.com/SergioBenitez/proc-macro2-diagnostics/commit/27be55a8b4b45ee615b54cd87e45c516f4317dd5. Thank you again.