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