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.

Sign up for GitHub

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

Jump to bottom

Drop maven 4 to alpha 4 / Exclude plexus container default #609

Merged

hazendaz merged 2 commits into spotbugs:spotbugs from hazendaz:spotbugs

Jun 19, 2023

Member

hazendaz commented Jun 19, 2023

In regards to maven 4

Alpha 5 seems to have regression that was resolve here [MNG-7648] Fix locationTracking in DefaultModelBuildingRequest copy constructor. apache/maven#940

In regards to plexus container default

Per Uses deprecated maven components #589, usage was reported. This comes from doxia logging api. After looking, it does not appear doxia logging api directly uses it at all. Further all I could find on doxia 1 was usage in unit tests. Its deleted from doxia 2. So this does to some degree indicate we possibly make more items provided. But for now, simply removed it out.

hazendaz added 2 commits

June 19, 2023 18:33


          [GHA] Drop to maven 4 alpha 4 to see if issue is a regression

a979713


          [pom] Exclude plexus container default

d78ff3f

hazendaz self-assigned this

hazendaz merged commit 7f24ea2 into spotbugs:spotbugs

18 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment