Skip to content

Conversation

@antonyhell
Copy link

No description provided.

Copy link
Member

@rexim rexim left a comment

Choose a reason for hiding this comment

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

Hello @antonyhell! Thank you very much for the contribution!

I have no idea what GitLab groups are since I don't use GitLab, but if it's something that you need support for I'm ok with that. This regex change looks a little bit scary to me since I have no idea if it breaks any existing GitLab functionality. Could we please have a couple of very simple unit test cases with and without GitLab groups? Nothing fancy. Just something simple and quick to sanity check the changes.

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