Skip to content

ci: trigger split repo tests on auth, gax, and shared-deps changes#12785

Merged
lqiu96 merged 3 commits intomainfrom
ci/trigger-split-repo-tests
Apr 14, 2026
Merged

ci: trigger split repo tests on auth, gax, and shared-deps changes#12785
lqiu96 merged 3 commits intomainfrom
ci/trigger-split-repo-tests

Conversation

@lqiu96
Copy link
Copy Markdown
Member

@lqiu96 lqiu96 commented Apr 13, 2026

Some "upstream" modules have a bit more impact and we want to test the impact of the changes across the modules inside the monorepo.

Changes in auth, gax, or shared-deps can trigger downstream unit tests

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@lqiu96 lqiu96 marked this pull request as ready for review April 13, 2026 23:45
@lqiu96 lqiu96 requested a review from a team as a code owner April 13, 2026 23:45
java-bigquery:
- 'java-bigquery/**'
- 'google-auth-library-java/**/*.java'
- 'sdk-platform-java/gax-java/**/*.java'
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.

Can we do sdk-platform-java/**? Changes in api-common and java-core may need to be verified as well. Same thing for other handwritten repos.

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.

I added a few more cases. I set it to be a bit selective with the trigger (Java code changes or dep changes) to avoid our CIs running extra since our CIs currently take a bit

@lqiu96 lqiu96 requested a review from blakeli0 April 14, 2026 00:39
@lqiu96 lqiu96 merged commit f4ddb3c into main Apr 14, 2026
105 of 110 checks passed
@lqiu96 lqiu96 deleted the ci/trigger-split-repo-tests branch April 14, 2026 03:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants