-
Notifications
You must be signed in to change notification settings - Fork 53
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
Add support to load onlyAnalyze classes/packages from specified file. #635
Conversation
… Excluded txt files for licensing.
@hazendaz I am not able to test negative scenario in my implementation. Every time newly introduced test onlyAnalyzeFileSource completed successfully. It would be great if you can point me to how to verify or if I am missing something. |
@Rahul-khandelwal Approved to run, will have to look at later, most likely this would be introduced sometime after spotbugs 4.8.0 is released. |
Sure @hazendaz , Please have a look when possible. Also let me know if I am missing something in testing. |
note: switched to master branch as target. Moving entire repo back to 'master'. The original split was due to upstream repo but its since retired and points here. So rather than have a one-off in spotbugs or for that matter in most repos, moving back to 'master'. Thanks. |
Thanks @Rahul-khandelwal . |
@hazendaz Thanks for merging. could you tell me any tentative dates for next release starting when I can use this feature? |
It's out now
Sent from my Verizon, Samsung Galaxy smartphone
Get Outlook for Android<https://aka.ms/AAb9ysg>
…________________________________
From: Rahul-khandelwal ***@***.***>
Sent: Tuesday, November 7, 2023 4:49:40 AM
To: spotbugs/spotbugs-maven-plugin ***@***.***>
Cc: Jeremy Landis ***@***.***>; Mention ***@***.***>
Subject: Re: [spotbugs/spotbugs-maven-plugin] Add support to load onlyAnalyze classes/packages from specified file. (PR #635)
@hazendaz<https://github.com/hazendaz> Thanks for merging. could you tell me any tentative dates for next release starting when I can use this feature?
—
Reply to this email directly, view it on GitHub<#635 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AAHODIYQ2L2V5HYQKTQNH2DYDH73JAVCNFSM6AAAAAA5FBCT52VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJYGE2TOMZRGI>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Released in 4.8.0.0 and 4.8.1.0 |
Use Case
When adding spotbugs to a large project, no every file can be fixed at once. In the gradual process of fixing all files, onlyAnalyze is a good tool to include selected files at very beginning of analysis.
Current implementation of parsing onlyAnalyze can't read classes/packages from file.
Having all classes/packages in pom.xml introduces a lot of boiler plate into pom file.
So it makes sense to have onlyAnalyze support for reading classes/packages from file(s).
Changes
Add support to load onlyAnalyze classes/packages from specified file.
Excluded txt files for licensing.