-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
gitlab: support for group whitelist #1436
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.
Thanks, that's a great addition.
Do we have tests for this connector that could be expanded?
Would you think factoring out some group-related helper used in all three connectors would make sense?
Yeah sorry, I haven't extended the test cases (I saw that there are tests for GitLab), I surely will. |
I was wrong, there are no GitLab connector tests yet, I will add a few :) |
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 this contribution! 😃
I'm not near a real computer, so I'll leave the merge for someone else.
How can we get this merged? :) |
I'll take a leap of faith... 😉 |
Thank you! 👍 |
gitlab: support for group whitelist, add tests
The GitLab connector was missing the group/org/team filtering as in BitBucket or GitHub connector. In this PR I would like to cure this absence.