From 726ffa338168e34ecf3d022966eff8c2c3cad858 Mon Sep 17 00:00:00 2001 From: Brian Ferris Date: Wed, 12 Oct 2022 10:57:31 -0700 Subject: [PATCH 1/2] Upgrade GitHub actions/ to use Node16 compatible versions. --- .github/workflows/acceptance_test.yml | 44 ++++++++++++------------ .github/workflows/docker.yml | 12 +++---- .github/workflows/end_to_end.yml | 20 +++++------ .github/workflows/formatting.yml | 10 +++--- .github/workflows/package_installers.yml | 8 ++--- .github/workflows/test_pack_doc.yml | 28 +++++++-------- .github/workflows/triage.yml | 4 +-- 7 files changed, 63 insertions(+), 63 deletions(-) diff --git a/.github/workflows/acceptance_test.yml b/.github/workflows/acceptance_test.yml index ab1f2d99ce..9baacc8feb 100644 --- a/.github/workflows/acceptance_test.yml +++ b/.github/workflows/acceptance_test.yml @@ -22,7 +22,7 @@ jobs: runs-on: ubuntu-latest steps: - name: Checkout Project - uses: actions/checkout@v2 + uses: actions/checkout@v3 with: # We need to fetch with a depth of 2 for pull_request so we can do HEAD^2 fetch-depth: 2 @@ -42,20 +42,20 @@ jobs: needs: pre_ci runs-on: ubuntu-latest steps: - - uses: actions/checkout@v2 + - uses: actions/checkout@v3 - uses: gradle/wrapper-validation-action@v1 pack-snapshot: needs: [ validate-gradle-wrapper ] runs-on: ubuntu-latest steps: - - uses: actions/checkout@v1 + - uses: actions/checkout@v3 - name: Set up JDK 11 - uses: actions/setup-java@v2 + uses: actions/setup-java@v3 with: java-version: '11' distribution: 'temurin' - name: Cache Gradle packages - uses: actions/cache@v2 + uses: actions/cache@v3 with: path: ~/.gradle/caches key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }} @@ -65,12 +65,12 @@ jobs: with: arguments: shadowJar - name: Persist gtfs-validator snapshot jar - uses: actions/upload-artifact@v2 + uses: actions/upload-artifact@v3 with: name: gtfs-validator-snapshot path: cli/build/libs/gtfs-validator-*-cli.jar - name: Persist comparator snapshot jar - uses: actions/upload-artifact@v2 + uses: actions/upload-artifact@v3 with: name: comparator-snapshot path: output-comparator/build/libs/output-comparator-*-cli.jar @@ -78,16 +78,16 @@ jobs: needs: [ validate-gradle-wrapper ] runs-on: ubuntu-latest steps: - - uses: actions/checkout@v1 + - uses: actions/checkout@v3 with: ref: master - name: Set up JDK 11 - uses: actions/setup-java@v2 + uses: actions/setup-java@v3 with: java-version: '11' distribution: 'temurin' - name: Cache Gradle packages - uses: actions/cache@v2 + uses: actions/cache@v3 with: path: ~/.gradle/caches key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }} @@ -97,7 +97,7 @@ jobs: with: arguments: shadowJar - name: Persist gtfs-validator jar from master branch - uses: actions/upload-artifact@v2 + uses: actions/upload-artifact@v3 with: name: gtfs-validator-master path: cli/build/libs/gtfs-validator-*-cli.jar @@ -107,7 +107,7 @@ jobs: runs-on: ubuntu-latest steps: - name: Checkout repository code - uses: actions/checkout@v1 + uses: actions/checkout@v3 - name: Install dependencies run: | pip install -r scripts/mobility-database-harvester/requirements.txt @@ -119,7 +119,7 @@ jobs: echo "::set-output name=matrix::$DATASETS" - name: Persist metadata if: always() - uses: actions/upload-artifact@v2 + uses: actions/upload-artifact@v3 with: name: datasets_metadata path: scripts/mobility-database-harvester/datasets_metadata @@ -131,14 +131,14 @@ jobs: strategy: matrix: ${{ fromJson(needs.fetch-urls.outputs.matrix) }} steps: - - uses: actions/checkout@v1 + - uses: actions/checkout@v3 - name: Download .jar file from master branch - uses: actions/download-artifact@v2 + uses: actions/download-artifact@v3 with: name: gtfs-validator-master path: gtfs-validator-master - name: Download latest changes .jar file from previous job - uses: actions/download-artifact@v2 + uses: actions/download-artifact@v3 with: name: gtfs-validator-snapshot path: gtfs-validator-snapshot @@ -149,7 +149,7 @@ jobs: env: OUTPUT_BASE: ${{ github.sha }} - name: Persist reports - uses: actions/upload-artifact@v2 + uses: actions/upload-artifact@v3 with: name: reports_all path: ${{ github.sha }}/output @@ -157,17 +157,17 @@ jobs: needs: [ get-reports ] runs-on: ubuntu-latest steps: - - uses: actions/checkout@v1 + - uses: actions/checkout@v3 - name: Download comparator .jar file from previous job - uses: actions/download-artifact@v2 + uses: actions/download-artifact@v3 with: name: comparator-snapshot - name: Retrieve reports from previous job - uses: actions/download-artifact@v2 + uses: actions/download-artifact@v3 with: name: reports_all - name: Retrieve gtfs latest versions from previous job - uses: actions/download-artifact@v2 + uses: actions/download-artifact@v3 with: name: datasets_metadata - name: Generate acceptance report test @@ -185,7 +185,7 @@ jobs: --run_id ${{github.run_id}} - name: Persist acceptance test reports if: always() - uses: actions/upload-artifact@v2 + uses: actions/upload-artifact@v3 with: name: acceptance_test_report path: acceptance-test-output diff --git a/.github/workflows/docker.yml b/.github/workflows/docker.yml index f623dfbb10..2eea1e5d2c 100644 --- a/.github/workflows/docker.yml +++ b/.github/workflows/docker.yml @@ -18,12 +18,12 @@ jobs: steps: - uses: actions/checkout@v3 - name: Set up JDK 11 - uses: actions/setup-java@v2 + uses: actions/setup-java@v3 with: java-version: '11' distribution: 'temurin' - name: Cache Gradle packages - uses: actions/cache@v2 + uses: actions/cache@v3 with: path: ~/.gradle/caches key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }} @@ -33,13 +33,13 @@ jobs: with: arguments: testReport - name: Persist **Passing** Java tests merged report - uses: actions/upload-artifact@v2 + uses: actions/upload-artifact@v3 with: name: Test report - Java path: build/reports/allTests/ - name: Persist **Failing** Java tests unmerged reports if: ${{ failure() }} - uses: actions/upload-artifact@v2 + uses: actions/upload-artifact@v3 with: name: Test report - Java path: | @@ -57,12 +57,12 @@ jobs: with: fetch-depth: 0 # need full clone so `./gradlew currentVersion` can search parents for older tags when needed - name: Set up JDK 11 - uses: actions/setup-java@v2 + uses: actions/setup-java@v3 with: java-version: '11' distribution: 'temurin' - name: Cache Gradle packages - uses: actions/cache@v2 + uses: actions/cache@v3 with: path: ~/.gradle/caches key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }} diff --git a/.github/workflows/end_to_end.yml b/.github/workflows/end_to_end.yml index 977a062e5c..5df264d63c 100644 --- a/.github/workflows/end_to_end.yml +++ b/.github/workflows/end_to_end.yml @@ -31,20 +31,20 @@ jobs: validate_gradle_wrapper: runs-on: ubuntu-latest steps: - - uses: actions/checkout@v2 + - uses: actions/checkout@v3 - uses: gradle/wrapper-validation-action@v1 pack-snapshot: needs: validate_gradle_wrapper runs-on: ubuntu-latest steps: - - uses: actions/checkout@v1 + - uses: actions/checkout@v3 - name: Set up JDK 11 - uses: actions/setup-java@v2 + uses: actions/setup-java@v3 with: java-version: '11' distribution: 'temurin' - name: Cache Gradle packages - uses: actions/cache@v2 + uses: actions/cache@v3 with: path: ~/.gradle/caches key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }} @@ -54,7 +54,7 @@ jobs: with: arguments: shadowJar - name: Persist gtfs-validator snapshot jar - uses: actions/upload-artifact@v2 + uses: actions/upload-artifact@v3 with: name: gtfs-validator-snapshot path: cli/build/libs/gtfs-validator-*-cli.jar @@ -62,7 +62,7 @@ jobs: runs-on: ubuntu-latest steps: - name: Checkout repository code - uses: actions/checkout@v1 + uses: actions/checkout@v3 - name: Install dependencies run: | pip install -r scripts/mobility-database-harvester/requirements.txt @@ -74,7 +74,7 @@ jobs: echo "::set-output name=matrix::$DATASETS" - name: Persist metadata if: always() - uses: actions/upload-artifact@v2 + uses: actions/upload-artifact@v3 with: name: datasets_metadata path: scripts/mobility-database-harvester/datasets_metadata @@ -86,9 +86,9 @@ jobs: strategy: matrix: ${{ fromJson(needs.fetch-urls.outputs.matrix) }} steps: - - uses: actions/checkout@v1 + - uses: actions/checkout@v3 - name: Download latest changes .jar file from previous job - uses: actions/download-artifact@v2 + uses: actions/download-artifact@v3 with: name: gtfs-validator-snapshot path: gtfs-validator-snapshot @@ -99,7 +99,7 @@ jobs: env: OUTPUT_BASE: ${{ github.sha }} - name: Persist reports - uses: actions/upload-artifact@v2 + uses: actions/upload-artifact@v3 with: name: reports_snapshot path: ${{ github.sha }}/output diff --git a/.github/workflows/formatting.yml b/.github/workflows/formatting.yml index e8cd067fce..07eb0e75fe 100644 --- a/.github/workflows/formatting.yml +++ b/.github/workflows/formatting.yml @@ -10,14 +10,14 @@ jobs: formatting: runs-on: ubuntu-latest steps: - - uses: actions/checkout@v2 + - uses: actions/checkout@v3 - name: Set up JDK 11 - uses: actions/setup-java@v2 + uses: actions/setup-java@v3 with: java-version: '11' distribution: 'temurin' - name: Cache Gradle packages - uses: actions/cache@v2 + uses: actions/cache@v3 with: path: ~/.gradle/caches key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }} @@ -28,11 +28,11 @@ jobs: arguments: verGJF - name: Comment PR if code does not comply to Google Java style guide if: ${{ failure() }} - uses: actions/github-script@v4 + uses: actions/github-script@v6 with: github-token: ${{secrets.GITHUB_TOKEN}} script: | - github.issues.createComment({ + github.rest.issues.createComment({ issue_number: context.issue.number, owner: context.repo.owner, repo: context.repo.repo, diff --git a/.github/workflows/package_installers.yml b/.github/workflows/package_installers.yml index 97c022f06f..09a7039781 100644 --- a/.github/workflows/package_installers.yml +++ b/.github/workflows/package_installers.yml @@ -17,7 +17,7 @@ jobs: validate_gradle_wrapper: runs-on: ubuntu-latest steps: - - uses: actions/checkout@v2 + - uses: actions/checkout@v3 - uses: gradle/wrapper-validation-action@v1 build_push: @@ -28,14 +28,14 @@ jobs: matrix: os: [ macos-latest, windows-latest, ubuntu-latest ] steps: - - uses: actions/checkout@v2 + - uses: actions/checkout@v3 with: # We need to download all tags so that the axion-release-plugin # can resolve the most recent version tag. fetch-depth: 0 - name: Set up JDK 17 - uses: actions/setup-java@v2 + uses: actions/setup-java@v3 with: # We need a recent version of Java with jpackage included. java-version: '17' @@ -107,7 +107,7 @@ jobs: --dest app/pkg/build/jpackage - name: "Upload Installer" - uses: actions/upload-artifact@v2 + uses: actions/upload-artifact@v3 with: name: Installer - ${{matrix.os}} path: | diff --git a/.github/workflows/test_pack_doc.yml b/.github/workflows/test_pack_doc.yml index 5b6a881b3e..2ce119a338 100644 --- a/.github/workflows/test_pack_doc.yml +++ b/.github/workflows/test_pack_doc.yml @@ -16,7 +16,7 @@ jobs: validate_gradle_wrapper: runs-on: ubuntu-latest steps: - - uses: actions/checkout@v2 + - uses: actions/checkout@v3 - uses: gradle/wrapper-validation-action@v1 test: needs: [ validate_gradle_wrapper ] @@ -26,14 +26,14 @@ jobs: java_version: [ 11, 17 ] os: [ ubuntu-latest, windows-latest ] steps: - - uses: actions/checkout@v1 + - uses: actions/checkout@v3 - name: Set up JDK ${{ matrix.java_version }} - uses: actions/setup-java@v2 + uses: actions/setup-java@v3 with: java-version: ${{ matrix.java_version }} distribution: 'temurin' - name: Cache Gradle packages - uses: actions/cache@v2 + uses: actions/cache@v3 with: path: ~/.gradle/caches key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }} @@ -43,13 +43,13 @@ jobs: with: arguments: testReport - name: Persist **Passing** tests merged report - Java ${{ matrix.java_version }} on ${{ matrix.os }} - uses: actions/upload-artifact@v2 + uses: actions/upload-artifact@v3 with: name: Test report - Java ${{ matrix.java_version }} on ${{ matrix.os }} path: build/reports/allTests/ - name: Persist **Failing** tests unmerged reports - Java ${{ matrix.java_version }} on ${{ matrix.os }} if: ${{ failure() }} - uses: actions/upload-artifact@v2 + uses: actions/upload-artifact@v3 with: name: Test report - Java ${{ matrix.java_version }} on ${{ matrix.os }} path: | @@ -66,14 +66,14 @@ jobs: java_version: [ 11, 17 ] os: [ ubuntu-latest, windows-latest ] steps: - - uses: actions/checkout@v1 + - uses: actions/checkout@v3 - name: Set up JDK ${{ matrix.java_version }} - uses: actions/setup-java@v2 + uses: actions/setup-java@v3 with: java-version: ${{ matrix.java_version }} distribution: 'temurin' - name: Cache Gradle packages - uses: actions/cache@v2 + uses: actions/cache@v3 with: path: ~/.gradle/caches key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }} @@ -83,7 +83,7 @@ jobs: with: arguments: shadowJar - name: Persist cli app jar - uses: actions/upload-artifact@v2 + uses: actions/upload-artifact@v3 with: name: Application - CLI executable - Java ${{ matrix.java_version }} JAR file -- ${{ matrix.os }} path: cli/build/libs/gtfs-validator-*-cli.jar @@ -95,14 +95,14 @@ jobs: java_version: [ 11, 17 ] os: [ ubuntu-latest, windows-latest ] steps: - - uses: actions/checkout@v1 + - uses: actions/checkout@v3 - name: Set up JDK ${{ matrix.java_version }} - uses: actions/setup-java@v2 + uses: actions/setup-java@v3 with: java-version: ${{ matrix.java_version }} distribution: 'temurin' - name: Cache Gradle packages - uses: actions/cache@v2 + uses: actions/cache@v3 with: path: ~/.gradle/caches key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }} @@ -112,7 +112,7 @@ jobs: with: arguments: aggregateJavadoc - name: Persist javadoc - uses: actions/upload-artifact@v2 + uses: actions/upload-artifact@v3 with: name: Documentation - javadoc - Java ${{ matrix.java_version }} - ${{ matrix.java_version }} path: build/docs/javadoc/ diff --git a/.github/workflows/triage.yml b/.github/workflows/triage.yml index 3e16079f6e..a869b64868 100644 --- a/.github/workflows/triage.yml +++ b/.github/workflows/triage.yml @@ -15,11 +15,11 @@ jobs: column: Requires investigation repo-token: ${{ secrets.TRIAGE_TOKEN }} - name: Comment issue - uses: actions/github-script@v4 + uses: actions/github-script@v6 with: github-token: ${{secrets.GITHUB_TOKEN}} script: | - github.issues.createComment({ + github.rest.issues.createComment({ issue_number: context.issue.number, owner: context.repo.owner, repo: context.repo.repo, From ff003885d77ad8f7f484adce7c162eb029ff38d6 Mon Sep 17 00:00:00 2001 From: Brian Ferris Date: Wed, 12 Oct 2022 12:51:52 -0700 Subject: [PATCH 2/2] Migrate from eskatos/gradle-command-action@v1 to gradle/gradle-build-action@v2 --- .github/workflows/acceptance_test.yml | 4 ++-- .github/workflows/docker.yml | 2 +- .github/workflows/end_to_end.yml | 2 +- .github/workflows/formatting.yml | 2 +- .github/workflows/test_pack_doc.yml | 6 +++--- 5 files changed, 8 insertions(+), 8 deletions(-) diff --git a/.github/workflows/acceptance_test.yml b/.github/workflows/acceptance_test.yml index 9baacc8feb..a49f312e8f 100644 --- a/.github/workflows/acceptance_test.yml +++ b/.github/workflows/acceptance_test.yml @@ -61,7 +61,7 @@ jobs: key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }} restore-keys: ${{ runner.os }}-gradle - name: Package cli app jar with Gradle - uses: eskatos/gradle-command-action@v1 + uses: gradle/gradle-build-action@v2 with: arguments: shadowJar - name: Persist gtfs-validator snapshot jar @@ -93,7 +93,7 @@ jobs: key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }} restore-keys: ${{ runner.os }}-gradle - name: Package cli app jar with Gradle - uses: eskatos/gradle-command-action@v1 + uses: gradle/gradle-build-action@v2 with: arguments: shadowJar - name: Persist gtfs-validator jar from master branch diff --git a/.github/workflows/docker.yml b/.github/workflows/docker.yml index 2eea1e5d2c..1b32bf0f14 100644 --- a/.github/workflows/docker.yml +++ b/.github/workflows/docker.yml @@ -29,7 +29,7 @@ jobs: key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }} restore-keys: ${{ runner.os }}-gradle - name: Run Java tests - uses: eskatos/gradle-command-action@v1 + uses: gradle/gradle-build-action@v2 with: arguments: testReport - name: Persist **Passing** Java tests merged report diff --git a/.github/workflows/end_to_end.yml b/.github/workflows/end_to_end.yml index 5df264d63c..6349558e57 100644 --- a/.github/workflows/end_to_end.yml +++ b/.github/workflows/end_to_end.yml @@ -50,7 +50,7 @@ jobs: key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }} restore-keys: ${{ runner.os }}-gradle - name: Package cli app jar with Gradle - uses: eskatos/gradle-command-action@v1 + uses: gradle/gradle-build-action@v2 with: arguments: shadowJar - name: Persist gtfs-validator snapshot jar diff --git a/.github/workflows/formatting.yml b/.github/workflows/formatting.yml index 07eb0e75fe..30b01cfffb 100644 --- a/.github/workflows/formatting.yml +++ b/.github/workflows/formatting.yml @@ -23,7 +23,7 @@ jobs: key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }} restore-keys: ${{ runner.os }}-gradle - name: Check code compliance to google java format standards - uses: eskatos/gradle-command-action@v1 + uses: gradle/gradle-build-action@v2 with: arguments: verGJF - name: Comment PR if code does not comply to Google Java style guide diff --git a/.github/workflows/test_pack_doc.yml b/.github/workflows/test_pack_doc.yml index 2ce119a338..a76403e84a 100644 --- a/.github/workflows/test_pack_doc.yml +++ b/.github/workflows/test_pack_doc.yml @@ -39,7 +39,7 @@ jobs: key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }} restore-keys: ${{ runner.os }}-gradle - name: Run tests on Java ${{ matrix.java_version }} and ${{ matrix.os }} - uses: eskatos/gradle-command-action@v1 + uses: gradle/gradle-build-action@v2 with: arguments: testReport - name: Persist **Passing** tests merged report - Java ${{ matrix.java_version }} on ${{ matrix.os }} @@ -79,7 +79,7 @@ jobs: key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }} restore-keys: ${{ runner.os }}-gradle - name: Package cli app jar with Gradle - uses: eskatos/gradle-command-action@v1 + uses: gradle/gradle-build-action@v2 with: arguments: shadowJar - name: Persist cli app jar @@ -108,7 +108,7 @@ jobs: key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }} restore-keys: ${{ runner.os }}-gradle - name: Build Javadoc - uses: eskatos/gradle-command-action@v1 + uses: gradle/gradle-build-action@v2 with: arguments: aggregateJavadoc - name: Persist javadoc