MartinKanters commented on pull request #374:
URL: https://github.com/apache/maven/pull/374#issuecomment-690445048


   > Is it possible to add a scenario for this in the DefaultGraphBuilderTest?
   
   Yes, it is possible, but would require a multi-layered submodule to be 
tested efficiently. We have introduced that in 
https://github.com/apache/maven/pull/373. 
   My preference is to include this scenario in that test once that PR is 
merged (or include the test scenario in that PR if this one is merged sooner).
   For now it is covered by integration-tests, which should be good enough I 
think. What do you think?


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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to