Skip to content

refactor: rewrite the permissions checker as a class#558

Merged
adriguy merged 5 commits intomasterfrom
feat/roles#07/request-new-permissions-format
Nov 26, 2020
Merged

refactor: rewrite the permissions checker as a class#558
adriguy merged 5 commits intomasterfrom
feat/roles#07/request-new-permissions-format

Conversation

@adriguy
Copy link
Contributor

@adriguy adriguy commented Nov 25, 2020

Pull Request checklist:

  • Write an explicit title for the Pull Request, following Conventional Commits specification
  • Test manually the implemented changes
  • Review my own code (indentation, syntax, style, simplicity, readability)
  • Wonder if you can improve the existing code

@adriguy adriguy assigned VincentMolinie and unassigned adriguy Nov 25, 2020
@adriguy adriguy force-pushed the feat/roles#07/request-new-permissions-format branch from 5c99cfa to 9293914 Compare November 25, 2020 17:30
@adriguy adriguy force-pushed the feat/roles#07/request-new-permissions-format branch from d0bfddf to b20ca66 Compare November 26, 2020 14:31
@adriguy adriguy assigned VincentMolinie and unassigned adriguy Nov 26, 2020
@qlty-cloud-legacy
Copy link

Code Climate has analyzed commit b20ca66 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 95.5% (49% is the threshold).

This pull request will bring the total coverage in the repository to 50.8%.

View more on Code Climate.

Copy link
Member

@VincentMolinie VincentMolinie left a comment

Choose a reason for hiding this comment

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

Let's merge this ! Good job 👏

@adriguy adriguy merged commit 3ae641c into master Nov 26, 2020
@adriguy adriguy deleted the feat/roles#07/request-new-permissions-format branch November 26, 2020 15:05
@forest-bot
Copy link
Member

🎉 This PR is included in version 7.7.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@forest-bot
Copy link
Member

🎉 This PR is included in version 8.0.0-beta.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments