-
-
Notifications
You must be signed in to change notification settings - Fork 24
Upgrade checkstyle to 12.3.0 #494
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
@wborn, thanks! Where do we define the version and source of the unleash plugin? Is one from the infrastructure-repo used? (Sorry, cannot see the connection yet. Is it pulled in in the Jenkins job only?) |
|
It's in the super POM: https://github.com/openhab/infrastructure/blob/584bc58b00260c73613c51f13bfb55cf784ba3c3/pom.xml#L59 We probably want to override the version and test it in the sandbox build first, before going through the hassle of releasing the super POM and finding out there are issues. We also need to make sure it works with 4.3.x, 5.0.x branches or release a super POM with a version outside of the ranges used by those branches. |
37b94b3 to
e35c1f6
Compare
|
PR can be adapted to 12.3.0 |
|
Yes, already done locally, but waiting for #535. |
e35c1f6 to
dca494e
Compare
dca494e to
b7c285d
Compare
Upgrade checkstyle from 10.26.1 to 12.3.0 Changlog: https://github.com/checkstyle/checkstyle/releases/ Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>
b7c285d to
dd3a19b
Compare
Upgrade checkstyle from 10.26.1 to 12.3.0
Changlog:
https://github.com/checkstyle/checkstyle/releases/
Depends on #535
Outdated:
@wborn, @kaikreuzer
Checkstyle 11 requires Java 17.
How do we proceed?
I cannot remember right now why we still require Java 11 here. Can you pls. give me a hint?