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

Update CLI filters with excludes option #22

Merged
merged 1 commit into from
Mar 31, 2024
Merged

Conversation

catatsuy
Copy link
Owner

@catatsuy catatsuy commented Mar 31, 2024

This pull request includes changes in the cli/cli.go and cli/cli_test.go files. The changes mainly revolve around renaming the notFilters member of the CLI struct to excludes. This renaming has been propagated throughout the codebase, affecting several methods and their corresponding tests.

Here are the most important changes:

Renaming notFilters to excludes:

  • cli/cli.go: The notFilters member in the CLI struct has been renamed to excludes. This change affects the Run, parseFlags, validateInput, and filterProcess methods, where all instances of notFilters have been replaced with excludes. [1] [2] [3] [4] [5]

Updating the tests:

  • cli/cli_test.go: All test cases that used the -not-filter flag have been updated to use the -exclude flag. This includes changes in TestRun_success and TestFilterProcess. [1] [2] [3] [4]

@catatsuy catatsuy merged commit 8740fca into main Mar 31, 2024
1 check passed
@catatsuy catatsuy deleted the feature_use_exclude branch March 31, 2024 04:20
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