feat: replace run.skip-xxx
options by issues.exclude-xxx
options
#4509
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The option name
skip-xxx
is a source of confusion.And to increase the confusion, those options are inside the section
run
.This sample of issues illustrate the problem: #1912, #2748, #4084, #3775, #1832, #4360
My proposal is move and rename those options:
run.skip-dirs
deprecation: replaced byissues.exclude-dirs
run.skip-dirs-use-default
deprecation: replaced byissues.exclude-dirs-use-default
run.skip-files
deprecation: replaced byissues.exclude-files
This is a non-breaking change: the
skip-xxx
will still work but with a warning, andissues.exclude-xxx
options do the same things asskip-xxx
.log output
Using a flag:
Using the file:
one commit, one change.