Skip to content

Commit

Permalink
Move the spotbugs check out of test_npm_8 and into `test_nomaven_11…
Browse files Browse the repository at this point in the history
…` because `test_npm_8` is failing repeatedly on CI due to resource exhaustion.
  • Loading branch information
nedtwigg committed Dec 6, 2021
1 parent 929e4af commit c003858
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 11 deletions.
21 changes: 11 additions & 10 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,15 @@ anchors:
path: lib-extra/build/test-results/test
- store_test_results:
path: plugin-gradle/build/test-results/test
- store_artifacts:
path: lib/build/spotbugs
- store_artifacts:
path: lib-extra/build/spotbugs
- store_artifacts:
path: testlib/build/spotbugs
- store_artifacts:
path: plugin-gradle/build/spotbugs

jobs:
# gradlew spotlessCheck assemble testClasses
assemble_testClasses:
Expand Down Expand Up @@ -108,22 +117,14 @@ jobs:
- store_test_results:
path: plugin-gradle/build/test-results/NpmTest
- run:
name: gradlew check
command: export SPOTLESS_EXCLUDE_MAVEN=true && ./gradlew check --build-cache
name: gradlew test
command: export SPOTLESS_EXCLUDE_MAVEN=true && ./gradlew test --build-cache
- store_test_results:
path: testlib/build/test-results/test
- store_test_results:
path: lib-extra/build/test-results/test
- store_test_results:
path: plugin-gradle/build/test-results/test
- store_artifacts:
path: lib/build/spotbugs
- store_artifacts:
path: lib-extra/build/spotbugs
- store_artifacts:
path: testlib/build/spotbugs
- store_artifacts:
path: plugin-gradle/build/spotbugs
test_windows:
executor:
name: win/default
Expand Down
2 changes: 1 addition & 1 deletion gradle/java-setup.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ tasks.named('spotbugsTest') {
}
tasks.named('spotbugsMain') {
// only run on Java 8 (no benefit to running twice)
enabled = org.gradle.api.JavaVersion.current() == org.gradle.api.JavaVersion.VERSION_1_8
enabled = org.gradle.api.JavaVersion.current() == org.gradle.api.JavaVersion.VERSION_11
reports {
html.enabled = true
}
Expand Down

0 comments on commit c003858

Please sign in to comment.