Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add zapFile rule type #49

Closed
wants to merge 1 commit into from
Closed

Add zapFile rule type #49

wants to merge 1 commit into from

Conversation

acourtneybrown
Copy link

Analogous to zap rule type, which only allows for omitting .class files, this change adds a zapFile type that will omit any file based on the path and wildcarding support available to other rules.

Fixes: #48

Analogous to `zap` rule type, which only allows for omitting `.class` files,
this change adds a `zapFile` type that will omit any file based on the
path and wildcarding support available to other rules.

Fixes: #48
@acourtneybrown acourtneybrown closed this by deleting the head repository Feb 13, 2024
@johnynek
Copy link
Collaborator

Sorry I didn't comment on this PR. I noticed the PR was red in CI and didn't check back.

I did appreciate your contribution and I think it would be nice to have general file processing abilities in the code.

@acourtneybrown
Copy link
Author

@johnynek I've moved on from the company where this change was relevant, so it's no longer "my problem to worry about"™️. I was just cleaning up things that I was not going to come back to and the PR closure had nothing to do with the repo codeowners 😄 .

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to zap non-.class files
2 participants