You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@maven.apache.org by GitBox <gi...@apache.org> on 2023/01/17 07:16:45 UTC

[GitHub] [maven] gnodet commented on pull request #954: [MNG-7629] Change reactor reader to copy packaged artifacts and reuse them across builds if needed

gnodet commented on PR #954:
URL: https://github.com/apache/maven/pull/954#issuecomment-1384932632

   
   > Left a couple of questions.
   > 
   > Also, I wonder if this change allows us to remove the fallbacks that were previously on line 150 - 159. Since those lines aren't changed, it seems GitHub doesn't allow me to comment on the spot :-\.
   
   Good question. I think part of it could be removed, but we need to keep the `determineBuildOutputDirectoryForArtifact` in order to be able to partially run mvn test.
   
   Another possible strategy instead of copying the artifacts would be to only record their paths.


-- 
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-unsubscribe@maven.apache.org

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