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

Patch for filtering #50

Closed
wants to merge 1 commit into from
Closed

Patch for filtering #50

wants to merge 1 commit into from

Conversation

wsnyder
Copy link
Contributor

@wsnyder wsnyder commented Jun 4, 2020

Previously the -C filtering was in a different code path and didn't have the EXCLs processed. This fixes that. Most of this is just moving code to a function, which I humbly think it makes more sense, as then it reads either combined or non-combined code, then filters with common code.

Note this now would require -X to be backward compatible with the old -C, you might want to change this so that -X is the default if -C is used, then I need a new flag to turn "on" exclusion markers with -C.

@wsnyder
Copy link
Contributor Author

wsnyder commented Jun 4, 2020

I'm presuming you'll use this as an example rather than merging, so not cleaning up the test failure.

@wsnyder
Copy link
Contributor Author

wsnyder commented Jun 4, 2020

Main code updated & this is no longer relevant.

@wsnyder wsnyder closed this Jun 4, 2020
@wsnyder wsnyder deleted the pull-issue-48-patch branch June 5, 2020 10:23
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.

1 participant