Shreekar-cloud commented on PR #13601:
URL: https://github.com/apache/maven/pull/13601#issuecomment-6306033974

   Hi @abdulhannansyed548-cmd, thanks for building on this and for waiting. I 
applied the LOAD_PROJECT approach on top of your head (aeb6dc1e) and pushed it 
here:
   
   https://github.com/Shreekar-cloud/maven/tree/fix-13395-load-project
   
   What it does:
   - Adds RequestType.LOAD_PROJECT. It gets the project treatment (local parent 
lookup, profile activation) but is parsed and validated leniently.
   - Reactor and standalone builds stay BUILD_EFFECTIVE. A POM file with 
processPlugins on becomes BUILD_EFFECTIVE if the validation level is STRICT, 
and LOAD_PROJECT otherwise.
   - Drops the getValidationLevel() API additions, so CacheConfigurationTest 
and AbstractMavenProjectTestCase go back to their originals.
   - Keeps your regression test, with the processPlugins loop reduced to true.
   
   Testing: ProjectBuilderTest#testLocalParentResolutionWithValidationLevels 
passes, and the maven-core, model and cache tests pass (259 run, 0 failures, 1 
skipped).
   
   How would you like to handle it? You can take these changes into your PR, or 
I can open a separate PR that credits you as co-author so the maintainers can 
pick one. Either works for me. I'll wait for your answer before opening 
anything.


-- 
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