Skip to content
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

Handle CODEOWNERS with a double asterix and a leading slash #805

Merged
merged 2 commits into from Nov 19, 2019

Conversation

@zegl
Copy link
Contributor

zegl commented Nov 19, 2019

This fixes #804

There seems to be a few testcases that incorrectly sets a leading slash when matching filepaths, but as far as I can see, no leading slash is set in production environments.

@notriddle

This comment has been minimized.

Copy link
Member

notriddle commented Nov 19, 2019

bors r+

bors bot added a commit that referenced this pull request Nov 19, 2019
Merge #805
805: Handle CODEOWNERS with a double asterix and a leading slash r=notriddle a=zegl

This fixes #804

There seems to be a few testcases that incorrectly sets a leading slash when matching filepaths, but as far as I can see, no leading slash is set in production environments.

Co-authored-by: Gustav Westling <gustav.westling@tink.se>
notriddle added a commit to bors-ng/bors-ng.github.io that referenced this pull request Nov 19, 2019
Add @zegl to new contributors
@bors

This comment has been minimized.

Copy link
Contributor

bors bot commented Nov 19, 2019

Build succeeded

@bors bors bot merged commit 89bb1b7 into bors-ng:master Nov 19, 2019
3 checks passed
3 checks passed
Community-TC (pull_request) TaskGroup: success
Details
Travis CI - Pull Request Build Passed
Details
bors Build succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.