search

ksprojects / zkcopy

ZooKeeper copy utililty
Apache License 2.0
240 stars 96 forks source link

WhitespaceAround errors in Writer.java #26

Closed zbukhari-apex closed 2 months ago

zbukhari-apex commented 2 months ago

src/main/java/com/github/ksprojects/zkcopy/writer/Writer.java

[ERROR] /home/foobar/zkcopy/src/main/java/com/github/ksprojects/zkcopy/writer/Writer.java:114:55: WhitespaceAround: '{' is not preceded with whitespace. [WhitespaceAround]
[ERROR] /home/foobar/zkcopy/src/main/java/com/github/ksprojects/zkcopy/writer/Writer.java:117:64: WhitespaceAround: '{' is not preceded with whitespace. [WhitespaceAround]
[ERROR] /home/foobar/zkcopy/src/main/java/com/github/ksprojects/zkcopy/writer/Writer.java:119:26: WhitespaceAround: 'else' is not preceded with whitespace. [WhitespaceAround]
[ERROR] /home/foobar/zkcopy/src/main/java/com/github/ksprojects/zkcopy/writer/Writer.java:119:26: WhitespaceAround: '}' is not followed by whitespace. Empty blocks may only be represented as {} when not part of a multi-block statement (4.1.3) [WhitespaceAround]

Easily fixed. I can submit a PR if desired. This is on Ubuntu Jammy (i.e. 22.04) using maven 3.6.3-5 and OpenJDK 11 (i.e. openjdk-11-jre-headless:amd64 11.0.23+9-1ubuntu1~22.04.1)

After fixing that and running it again, it completed but failed on tests. Separate issue I believe but just mentioning it here.

zbukhari-apex commented 2 months ago

pom.xml says 0.2.1-SNAPSHOT