jbertram commented on PR #4699:
URL:
https://github.com/apache/activemq-artemis/pull/4699#issuecomment-1839483145
@MrEasy, you've got a handful of checkstyle violations:
```
[INFO] There are 8 errors reported by Checkstyle 10.11.0 with
/home/runner/work/activemq-artemis/activemq-artemis/artemis-core-client/../etc/checkstyle.xml
ruleset.
Error:
src/main/java/org/apache/activemq/artemis/core/client/impl/ClientSessionFactoryImpl.java:[196,4]
(blocks) LeftCurly: '{' at column 4 should be on the previous line.
Error:
src/main/java/org/apache/activemq/artemis/core/client/impl/ServerLocatorImpl.java:[194,4]
(blocks) LeftCurly: '{' at column 4 should be on the previous line.
Error:
src/main/java/org/apache/activemq/artemis/core/client/impl/ServerLocatorImpl.java:[197,7]
(blocks) RightCurly: '}' at column 7 should be on the same line as the next
part of a multi-block statement (one that directly contains multiple blocks:
if/else-if/else, do/while or try/catch/finally).
Error:
src/main/java/org/apache/activemq/artemis/core/client/impl/ServerLocatorImpl.java:[204,7]
(blocks) RightCurly: '}' at column 7 should be on the same line as the next
part of a multi-block statement (one that directly contains multiple blocks:
if/else-if/else, do/while or try/catch/finally).
Error:
src/main/java/org/apache/activemq/artemis/core/client/impl/ServerLocatorImpl.java:[209,10]
(blocks) LeftCurly: '{' at column 10 should be on the previous line.
Error:
src/main/java/org/apache/activemq/artemis/core/client/impl/ServerLocatorImpl.java:[212,13]
(blocks) LeftCurly: '{' at column 13 should be on the previous line.
Error:
src/main/java/org/apache/activemq/artemis/core/client/impl/ServerLocatorImpl.java:[220,10]
(blocks) RightCurly: '}' at column 10 should be on the same line as the next
part of a multi-block statement (one that directly contains multiple blocks:
if/else-if/else, do/while or try/catch/finally).
Error:
src/main/java/org/apache/activemq/artemis/core/client/impl/ServerLocatorImpl.java:[228,13]
(blocks) LeftCurly: '{' at column 13 should be on the previous line.
```
You can check this yourself locally if you want via:
```sh
mvn -Pdev install
```
See the [hacking
guide](https://activemq.apache.org/components/artemis/documentation/hacking-guide/#using-the-dev-profile)
for more details.
--
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: gitbox-unsubscr...@activemq.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
