desruisseaux commented on PR #2219: URL: https://github.com/apache/maven/pull/2219#issuecomment-2772558151
Alternatively, we can make the constructor package-private and add another constructor without the `PathModularizationCache` argument, which would delegate to the package-private constructor with a `null` argument. A few months ago, a started a branch where I tried to reduce the visibility of some Maven internal classes. It required moving some classes to a common package (in this case, we would need to move `DefaultProjectBuilder`). I did not explored further because of conflict with other changes, but generally I would prefer going in the direction of removing some `public` items instead of adding more. -- 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: issues-unsubscr...@maven.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
