-
Notifications
You must be signed in to change notification settings - Fork 72
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
Updating --include-path-patterns and --exclude-path-patterns to use a table of tables #289
Conversation
) | ||
self.global_options.exclude_path_patterns = ("bar/",) |
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.
Same comment as the test above.
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.
Pretty much one change to relax unintended restriction that needlessly crosses multiple configuration files.
@rbailey-godaddy I have created an issue to address the last few review comments as part of another PR after 3.0 is released |
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.
LGTM (keeping in mind the new backlog story)
Clean up `included_paths` and `excluded_paths` with basically the same logic used for godaddy#290.
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.
🎉
To help us get this pull request reviewed and merged quickly, please be sure to include the following items:
PR Type
What kind of change does this PR introduce?
Backward Compatibility
Is this change backward compatible with the most recently released version? Does it introduce changes which might change the user experience in any way? Does it alter the API in any way?
Issue Linking
closes #257
What's new?
--include-path-patterns
and--exclude-path-patterns
with an option to specify the reason.Ex: include-path-patterns = [{path-pattern="path-pattern", reason="reason to include path patterns"}]