Skip to content

Commit

Permalink
test: rename full-convergence-check.yaml to bom-content-test (#6366)
Browse files Browse the repository at this point in the history
* test: rename full-convergence-check.yaml to bom-content-test

Removing full-dependencies-convergence check because, while it's desirable to have full-convergence, we do not act on the failures and there's no harm to customers to have slight differences in dependencies.

* renamed

* renamed to yaml
  • Loading branch information
suztomo committed May 8, 2024
1 parent f3c611a commit 6e38599
Showing 1 changed file with 0 additions and 18 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -47,21 +47,3 @@ jobs:
mvn -B -V -ntp verify -Dtest="BomContentAssertionsTest"
working-directory: tests
full-dependencies-convergence:
runs-on: ubuntu-latest
if: github.repository_owner == 'googleapis' && github.head_ref == 'release-please--branches--main'
steps:
- uses: actions/checkout@v4
- uses: actions/setup-java@v4
with:
distribution: zulu
java-version: 8
- run: java -version
- name: Install BOMs
run: |
mvn -B -V -ntp install
- name: Validate dependency convergence of library dependencies in the Google Cloud BOM
run: |
mvn -B -V -ntp validate
working-directory: tests/dependency-convergence

0 comments on commit 6e38599

Please sign in to comment.