-
Notifications
You must be signed in to change notification settings - Fork 5
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
[68] added utils.check_roles_enabled helper #69
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your contribution! See several comments below.
My main concern is the support of role dependencies. I hope you'll help me to clarify this question.
There was a problem hiding this 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 patch, I think we still can improve tests a bit and then I'll merge it
It also look like you tests had failed for some reason. We'll need to fix it before merging. |
There was a problem hiding this 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 updates! You can merge the changes into a single commit as we discussed above (after adding the last missing newline)
a513c2f
to
1d223d0
Compare
1d223d0
to
6da855e
Compare
This helper check are provided roles enabled on instance.
It might be very usefull if you want to run migration code only on specific roles.
Closes #68