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

Extend syntax to allow ignore annotation on tests - with tests #10

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

Conversation

svenfoo
Copy link

@svenfoo svenfoo commented Oct 16, 2020

This is the same as #8 , but it also adds tests for the MacroAttributes parser. I haven't managed to write the tests against the old syn API, thus this merge request also includes #9.

@svenfoo
Copy link
Author

svenfoo commented Oct 18, 2020

Just realized that the change became larger than intended because I ran cargo fmt on the whole workspace. If that is a problem, just let me know and I'll redo the change without the needless reformatting.

@ollien
Copy link

ollien commented Nov 22, 2023

Would love to see this merged :) was a bit disappointed to see this didn't work in the way I expected

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