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

Add option to allow unowned patterns #113

Merged
merged 4 commits into from
Feb 12, 2022
Merged

Conversation

mszostok
Copy link
Owner

@mszostok mszostok commented Feb 10, 2022

Description

Changes proposed in this pull request:

  • Add option to allow unowned patterns
    • Move this check under owner as from the syntax point of view it's correct as properly pointed out by @joshua-cannon-techlabs.
  • Update integration test

Related issue(s)

Fix #94

@mszostok mszostok added enhancement New feature or request WIP Work In Progress labels Feb 10, 2022
@mszostok mszostok force-pushed the opt-for-unowned-pattern branch 3 times, most recently from ff0b311 to 2728830 Compare February 12, 2022 17:18
@mszostok mszostok removed the WIP Work In Progress label Feb 12, 2022
@mszostok mszostok merged commit 5367f8a into main Feb 12, 2022
@mszostok mszostok deleted the opt-for-unowned-pattern branch February 12, 2022 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Syntax Checker errors with "missing owner" on valid CODEOWNERS
1 participant