kriegaex commented on PR #204:
URL: 
https://github.com/apache/maven-javadoc-plugin/pull/204#issuecomment-1853641129

   > The reason you see both `reports/` _and_ `site/` is that some ITs use 
standalone invocation and others site invocation.
   
   I know that. You misread my comment. But before your force-push, there was 
**only** `site`, and I reviewed the old commit separately first. Only later, I 
noticed the new commit, which now takes `reports` into account, too. This is 
why I wrote "LTGM" (looks good to me) on top of my post.


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