Skip to content

Support negated onlyAnalyze items#2754

Merged
hazendaz merged 1 commit intospotbugs:masterfrom
rovarga:skip-class
Dec 9, 2023
Merged

Support negated onlyAnalyze items#2754
hazendaz merged 1 commit intospotbugs:masterfrom
rovarga:skip-class

Conversation

@rovarga
Copy link
Copy Markdown
Contributor

@rovarga rovarga commented Dec 7, 2023

Having the ability to exclude only specific files is useful where a
project needs to be partially analyzed and it is the exclusions that are
known, not the inclusions.

Update -onlyAnalyze to recognize items starting with ! as exclusions,
allowing a lot more flexibility in specification of which classes are
included.

Signed-off-by: Robert Varga robert.varga@pantheon.tech

Before opening a 'pull request'

  • Search existing issues and pull requests to see if the issue was already discussed.
  • Check our discussions to see if the issue was already discussed.
  • Check for specific project we support to raise the issue on, under spotbugs
  • Do not open intellij plugin issues here, open them at intellij-plugin *

Make sure these boxes are checked before submitting your PR -- thank you!

  • Added an entry into CHANGELOG.md if you have changed SpotBugs code

@rovarga rovarga force-pushed the skip-class branch 2 times, most recently from 42a8730 to a36be19 Compare December 7, 2023 21:24
Having the ability to exclude only specific files is useful where a
project needs to be partially analyzed and it is the exclusions that are
known, not the inclusions.

Update -onlyAnalyze to recognize items starting with ! as exclusions,
allowing a lot more flexibility in specification of which classes are
included.

Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
@hazendaz hazendaz self-assigned this Dec 9, 2023
@hazendaz
Copy link
Copy Markdown
Member

hazendaz commented Dec 9, 2023

@rovarga Nice! Will apply shortly.

@hazendaz hazendaz merged commit e720004 into spotbugs:master Dec 9, 2023
@rovarga rovarga deleted the skip-class branch December 11, 2023 06:21
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.

3 participants