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

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2020/12/03 15:22:48 UTC

[GitHub] [pulsar] BewareMyPower commented on pull request #8819: [CI] Fix maven install failure caused by spotbugs check

BewareMyPower commented on pull request #8819:
URL: https://github.com/apache/pulsar/pull/8819#issuecomment-738075290


   By the way, for pulsar committers, I suggested that if a PR contains changes **that may affect the basic build workflow**, it should be reviewed carefully. After it's merged, the other PRs should be checked if they rebased to the latest commit. If not, we should remind authors to rebase to the latest commit.
   
   If I remember correctly, the similar problem happened not long ago. And there're a lot of spotbugs plugins to be added to each module. If PRs were not reviewed carefully, the problem would be endless.


----------------------------------------------------------------
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:
users@infra.apache.org