rzo1 commented on PR #2123: URL: https://github.com/apache/stormcrawler/pull/2123#issuecomment-5589870623
Third `rat` failure in a row here: Checkstyle indentation on the 6th, `CustomImportOrder` on the 7th, and now the formatter on `SpoutStatusStreamWiringTest.java` ([run 34248381078](https://github.com/apache/stormcrawler/actions/runs/34248381078)). `build` is `needs: rat`, so the test matrix has never run on this PR and I still have nothing to review against. Please check this locally *before* pushing. Every push needs a maintainer to manually approve the workflow run, so a push that dies on formatting costs someone else a round trip for zero information. Install the hook once, per README.md: ``` mvn clean install -Dskip.format.code=false ``` or at minimum run this and confirm it's clean before each push: ``` mvn -B -Prat -DskipTests verify -Dskip.format.code=false ``` Same applies to your other open PRs. -- 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]
