Skip to content

Commit

Permalink
ci: release=8 for JDK 9+ (#478)
Browse files Browse the repository at this point in the history
* ci: release=8 for JDK 9+

* ci: removing "dependencies (8)"
  • Loading branch information
suztomo authored Jun 10, 2022
1 parent 34993d5 commit 2468276
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 5 deletions.
1 change: 0 additions & 1 deletion .github/sync-repo-settings.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@ branchProtectionRules:
requiresStrictStatusChecks: false
# List of required status check contexts that must pass for commits to be accepted to matching branches.
requiredStatusCheckContexts:
- "dependencies (8)"
- "dependencies (11)"
- "lint"
- "clirr"
Expand Down
4 changes: 1 addition & 3 deletions .github/workflows/downstream-dependencies.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ jobs:
strategy:
fail-fast: false
matrix:
java: [8, 11]
java: [11]
repo:
- java-bigquery
- java-bigqueryconnection
Expand All @@ -27,5 +27,3 @@ jobs:
- run: sudo apt-get update -y
- run: sudo apt-get install libxml2-utils
- run: .kokoro/client-library-check.sh ${{matrix.repo}} dependencies
if: ${{matrix.java != '8' }}
# Always successful for Java 8, which we'll remove soon
13 changes: 12 additions & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -833,6 +833,17 @@
</plugins>
</build>
</profile>

<profile>
<!-- JDK 9+ has the "release" option to ensure the generated bytecode is
compatible with Java 8. Maven-compiler-plugin's "<target>1.8</target>" is
not sufficient. -->
<id>compiler-release-8</id>
<activation>
<jdk>[9,]</jdk>
</activation>
<properties>
<maven.compiler.release>8</maven.compiler.release>
</properties>
</profile>
</profiles>
</project>

0 comments on commit 2468276

Please sign in to comment.