-
Notifications
You must be signed in to change notification settings - Fork 39
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
Upgrade Checkstyle 10.12.5 -> 10.12.6 #911
Conversation
Suggested commit message:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This means we can also simplify the CheckStyle integration test for macOS :)
5bdf360
to
1b7023c
Compare
Looks good. No mutations were possible for these changes. |
e5b9062
to
f5885fa
Compare
Looks good. No mutations were possible for these changes. |
I'm curious what the result of running the integration test is right now. |
/integration-test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After they release 10.12.7 the init patch will be shorter as they use EPS from then 🤞🏻 .
f5885fa
to
14d8dc8
Compare
|
Looks good. No mutations were possible for these changes. |
And it will work on my machine because of this one: checkstyle/checkstyle#14131. |
This PR contains the following updates:
10.12.5
->10.12.6