Skip to content

Commit

Permalink
use node.js version 16 to fixed ci problem (#441)
Browse files Browse the repository at this point in the history
Co-authored-by: imbajin <[email protected]>
  • Loading branch information
wanganjuan and imbajin authored Mar 15, 2023
1 parent 4b6f074 commit b138651
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 4 deletions.
6 changes: 3 additions & 3 deletions .asf.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,9 +43,9 @@ github:
# strict means "Require branches to be up-to-date before merging".
strict: true
# contexts are the names of checks that must pass
contexts: []
# - build
# - Analyze (java)
contexts:
- check-license-header
- Analyze (java)
required_pull_request_reviews:
dismiss_stale_reviews: true
require_code_owner_reviews: false
Expand Down
6 changes: 5 additions & 1 deletion .github/workflows/codeql-analysis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,11 @@ jobs:
with:
distribution: 'zulu'
java-version: '8'


- name: Use Node.js 16
uses: actions/setup-node@v3
with:
node-version: '16'
# Initializes the CodeQL tools for scanning.
- name: Initialize CodeQL
uses: github/codeql-action/init@v2
Expand Down
9 changes: 9 additions & 0 deletions .github/workflows/license-checker.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,11 @@ jobs:
java-version: '11'
distribution: 'adopt'

- name: Use Node.js 16
uses: actions/setup-node@v3
with:
node-version: '16'

- name: License check(RAT)
run: |
mvn apache-rat:check -ntp
Expand All @@ -63,6 +68,10 @@ jobs:
with:
java-version: '11'
distribution: 'adopt'
- name: Use Node.js 16
uses: actions/setup-node@v3
with:
node-version: '16'
- name: mvn install
run: |
mvn install -DskipTests=true -ntp
Expand Down

0 comments on commit b138651

Please sign in to comment.