maximilian-novikov-db commented on pull request #597:
URL: https://github.com/apache/maven/pull/597#issuecomment-962897611


   > > This diff looks huge to me
   > 
   > yes surely there is some rework to make this diff easier to evaluate and 
choose to merge into a main branch for a "normal" Maven release I tried to move 
the whole "caching" package into a separate module a try to have a more visible 
separation of concerns, but 3 classes had dependencies on maven-core: 
MojoExecutionManager, ProjectUtils, CacheController. Then it's not so easy...
   > 
   > I chose to start by working on Modello migration #598, that will ease some 
parts: help welcome any other idea on finding a concrete step to go forward are 
welcome
   
   @hboutemy @michael-o we've prepared a change to the core which will allow to 
move the caching module to an extension, it's in testing now. We're going to 
raise a pr in a week.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to