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

refactor!: merge proc-macro into single crate and align implementation #50

Merged
merged 5 commits into from
Nov 27, 2023

Conversation

DDtKey
Copy link
Owner

@DDtKey DDtKey commented Nov 27, 2023

Description:

Merge proc-macro for actix-web-grants, poem-grants & rocket-grants into single crate to improve maintainability and get rid of repetitive changes.

Due to alignment some changes are included into this PR:

  • rename type to ty due to changes in syn2 (keywords are disallowed)
  • allow combination of authorities by all/any flag
  • rename secure to expr and allow to be combined
  • error attribute is available in all crates

Checklist:

  • Tests for the changes have been added (for bug fixes / features);
  • Docs have been added / updated (for bug fixes / features).
    • Partially, docs still something to improve

@DDtKey DDtKey merged commit d56a4b0 into main Nov 27, 2023
10 checks passed
@DDtKey DDtKey deleted the align-proc-macros branch November 27, 2023 20:38
@DDtKey DDtKey mentioned this pull request Nov 27, 2023
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.

1 participant