Skip to content

ci: removing dependencies (8) from required checks#479

Merged
suztomo merged 4 commits intogoogleapis:mainfrom
suztomo:downstream_jdk11
Jun 10, 2022
Merged

ci: removing dependencies (8) from required checks#479
suztomo merged 4 commits intogoogleapis:mainfrom
suztomo:downstream_jdk11

Conversation

@suztomo
Copy link
Copy Markdown
Member

@suztomo suztomo commented Jun 3, 2022

We do not have dependencies that differ depending on Java versions. Checking 'dependencies (11)' is enough.

We do not have dependencies that differ depending on Java versions
Checking 'dependencies (11)' is enough.
@suztomo suztomo requested a review from a team June 3, 2022 20:24
@product-auto-label product-auto-label Bot added the size: xs Pull request size is extra small. label Jun 3, 2022
@generated-files-bot
Copy link
Copy Markdown

Warning: This pull request is touching the following templated files:

  • .github/sync-repo-settings.yaml

@suztomo suztomo changed the title ci: removing dependencies (8) from required ci: removing dependencies (8) from required checks Jun 3, 2022
@suztomo suztomo mentioned this pull request Jun 3, 2022
@suztomo suztomo force-pushed the downstream_jdk11 branch from 0dd566f to b04e611 Compare June 9, 2022 21:55
- 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' }}
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This (and "dependencies (8)") will be removed in a subsequent pull request.

Comment thread owlbot.py Outdated
".github/release-please.yml",
".github/workflows/auto-release.yaml",
".github/workflows/samples.yaml",
".github/sync-repo-settings.yaml",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sync repo settings will not be edited by owlbot. It doesn't have to be in the exclusion list.

@suztomo suztomo added the automerge Merge the pull request once unit tests and other checks pass. label Jun 10, 2022
@suztomo suztomo merged commit 34993d5 into googleapis:main Jun 10, 2022
@suztomo suztomo deleted the downstream_jdk11 branch June 10, 2022 16:30
@gcf-merge-on-green gcf-merge-on-green Bot removed the automerge Merge the pull request once unit tests and other checks pass. label Jun 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: xs Pull request size is extra small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants