Skip to content

Remove FindBugs from reporting #159

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

Jump to bottom
Merged
merged 1 commit into from
Apr 26, 2023
Merged

Remove FindBugs from reporting #159

merged 1 commit into from
Apr 26, 2023

Conversation

slawekjaranowski
Copy link
Member

FindBugs is not maintain anymore ...

@slawekjaranowski
Remove FindBugs from reporting
b3c1202 
FindBugs is not maintain anymore ...
olamy
olamy approved these changes Apr 26, 2023
View reviewed changes
Copy link
Member

@olamy olamy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe we can remove even more reports (not sure someone reads them..)

@slawekjaranowski
Copy link
Member Author

maybe we can remove even more reports (not sure someone reads them..)

Yes I agree - it looks like its was copied from ASF maven parent
Many site of plexus projects not build correctly, broken links and so ... but we have many unusable reports (for me 😄 ) here

@olamy
Copy link
Member

olamy commented Apr 26, 2023

maybe we can remove even more reports (not sure someone reads them..)

Yes I agree - it looks like its was copied from ASF maven parent Many site of plexus projects not build correctly, broken links and so ... but we have many unusable reports (for me 😄 ) here

see that's why we do not need to use ASF poms 😎 ;)

@slawekjaranowski slawekjaranowski merged commit 03e23ed into master Apr 26, 2023
@slawekjaranowski slawekjaranowski deleted the rm-findbugs branch April 26, 2023 20:22
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@olamy olamy olamy approved these changes

Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@slawekjaranowski @olamy