echauchot commented on pull request #15218:
URL: https://github.com/apache/beam/pull/15218#issuecomment-891085988


   > @ibzib fixed everything except the package-info.java missing. I don't know 
how it is supposed to work with per-version source overrides. These overrides 
have the same java packages as the main source so the package-info file should 
be only in the main source packages otherwise they will be duplicated. Why is 
checkstyle complaining about not having package-info in source overrides 
whereas there are in corresponding main java packages?
   
   Found it:
   ```
    <!-- Checkstyle does not correctly detect package files across multiple 
source directories. -->
     <suppress checks="JavadocPackage" 
   ```


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