-
Notifications
You must be signed in to change notification settings - Fork 166
Add spotless auto-fix for PRs #1890
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
Add spotless auto-fix for PRs #1890
Conversation
145d080
to
39c97b8
Compare
39c97b8
to
787a9dd
Compare
- name: Spotless | ||
run: ./gradlew spotlessApply |
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.
nit: the workflow called spotless-check
is the one that runs spotlessApply
and the workflow called spotless-apply
doesn't. It's only slightly confusing, not a huge deal.
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.
yeah, it's named more for what you see in the github U/X, I'll probably make it all more generic and rename things again later
Spotless Check runs on PRs, generates the patch file, then Spotless Apply runs afterwards and applies the patch file
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 seems cool! Thanks.
5368714
to
41cb793
Compare
Basically the same as open-telemetry/opentelemetry-java-instrumentation#13872 + open-telemetry/opentelemetry-java-instrumentation#13879.