Cleanup Plexus Utils, convergence with Maven 4 step 1

mojohaus / versions

Versions Maven Plugin

https://www.mojohaus.org/versions/versions-maven-plugin/

Apache License 2.0

336 stars 267 forks source link

Cleanup Plexus Utils, convergence with Maven 4 step 1 #1169

Closed andrzejj0 closed 4 weeks ago

andrzejj0 commented 4 weeks ago

Clean up deprecated methods. Try to get rid as much as possible of Plexus Utils, see e.g. MNG-6825, MNG-7820.

This exercise focuses primarily on easy targets such as eliminating Plexus StringUtils in favour of Apache Commons etc.

The ultimate goal is to ease the upgrade to the upcoming changes from Maven 4.

slawekjaranowski commented 4 weeks ago

I think that we not need an issue where we immediately provide PR. Simply we can describe what we do in PR.

For me issue is useful when we want discus something before implementing or put on task list to do.

What do you think?

We can clarify it in README ...

andrzejj0 commented 4 weeks ago

Noted!

andrzejj0 commented 3 weeks ago

I think it does serve one important purpose though -- it lets us track progress on development. For example, I'm currently busy replacing ModifiedPOMXMLEventReader with a similar class based on Stax2, not using plexus xml anymore.