Skip to content

Conversation

@frederik-uni
Copy link

Description:

Instead of AuthorityGuard::new its now named AuthorityGuard::contains & AuthorityGuard::any & AuthorityGuard::all were added

Checklist:

  • Tests for the changes have been added (for bug fixes / features);
  • Docs have been added / updated (for bug fixes / features).

@DDtKey DDtKey changed the title add AuthorityGuard::contains, all, any feat(actix-web): add contains, all, any to AuthorityGuard Mar 23, 2025
Copy link
Owner

@DDtKey DDtKey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the contribution 🙏

I’ve left a couple of minor comments, please take a look.

@DDtKey
Copy link
Owner

DDtKey commented Mar 31, 2025

Could you apply rustfmt with nightly flag?

Co-authored-by: Artem Medvedev <i@ddtkey.com>
@DDtKey
Copy link
Owner

DDtKey commented Apr 6, 2025

Could you apply rustfmt and LGTM 👍

Thank you for the 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