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
Move spotless check to separate build task #1946
Changes from 3 commits
7080dda
6ceb726
aede9a9
60952f8
07e6a6a
e098079
f3e4584
f3657f4
39fa9b6
afa914d
4e04e76
4990c01
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -20,20 +20,17 @@ matrix: | |
- jdk: openjdk8 | ||
- jdk: openjdk8 | ||
env: SKIP_RELEASE=true MOCK_MAKER=mock-maker-inline | ||
- jdk: openjdk9 | ||
- jdk: openjdk11 | ||
env: SKIP_RELEASE=true | ||
- jdk: openjdk9 | ||
- jdk: openjdk11 | ||
env: SKIP_RELEASE=true MOCK_MAKER=mock-maker-inline | ||
- jdk: openjdk9 | ||
env: SKIP_RELEASE=true SIMULATE_JAVA11=true | ||
- jdk: openjdk10 | ||
- jdk: openjdk14 | ||
env: SKIP_RELEASE=true | ||
- jdk: openjdk10 | ||
- jdk: openjdk14 | ||
env: SKIP_RELEASE=true MOCK_MAKER=mock-maker-inline | ||
- jdk: openjdk11 | ||
env: SKIP_RELEASE=true | ||
- jdk: openjdk11 | ||
env: SKIP_RELEASE=true MOCK_MAKER=mock-maker-inline | ||
script: ./gradlew spotlessCheck | ||
after_success: true | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What does There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. To make sure we don't run the coverage report task, since that's not what spotless generates. I will add a clarifying comment. |
||
|
||
branches: | ||
#Don't build tags | ||
|
@@ -50,7 +47,7 @@ script: | |
# We are using && below on purpose | ||
# ciPerformRelease must run only when the entire build has completed | ||
# This guarantees that no release steps are executed when the build or tests fail | ||
- ./gradlew spotlessCheck && ./gradlew build idea -s && ./gradlew checkReproducibility && ./gradlew ciPerformRelease | ||
- ./gradlew build idea -s && ./gradlew checkReproducibility && ./gradlew ciPerformRelease | ||
|
||
after_success: | ||
#Generates coverage report: | ||
|
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.
Does
script: ./gradlew spotlessCheck
mean that on openjdk11 we only run "spotlessCheck" task?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.
It means that we have 1 task that only runs spotless, and that task runs on opendjk11. We still have the other openjdk11 builds to verify that the tests pass etc... I will add a comment for clarification.