Skip to content

build: move hermetic_library_generation filtering logic into the steps rather than on.pull_request#3410

Merged
BenWhitehead merged 3 commits intomainfrom
build/hermetic-config
Dec 2, 2025
Merged

build: move hermetic_library_generation filtering logic into the steps rather than on.pull_request#3410
BenWhitehead merged 3 commits intomainfrom
build/hermetic-config

Conversation

@BenWhitehead
Copy link
Copy Markdown
Collaborator

When filtering with on.pull_request the workflow will not report a status if it does not match. This causes contention with this workflow being declared as a required status. To sidestep this, move the matching logic into the steps of the workflow, similar to how we filter out PRs from forks.

@generated-files-bot
Copy link
Copy Markdown

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

  • .kokoro/build.sh

@product-auto-label product-auto-label Bot added size: s Pull request size is small. api: storage Issues related to the googleapis/java-storage API. labels Dec 1, 2025
@BenWhitehead BenWhitehead force-pushed the build/hermetic-config branch 4 times, most recently from 031e01d to 888f716 Compare December 1, 2025 21:35
…s rather than on.pull_request

When filtering with on.pull_request the workflow will not report a status if it does not match. This causes contention with this workflow being declared as a required status. To sidestep this, move the matching logic into the steps of the workflow, similar to how we filter out PRs from forks.
@BenWhitehead BenWhitehead force-pushed the build/hermetic-config branch from 888f716 to 34520a7 Compare December 1, 2025 21:38
@BenWhitehead BenWhitehead marked this pull request as ready for review December 1, 2025 22:00
@BenWhitehead BenWhitehead requested review from a team December 1, 2025 22:00
@BenWhitehead BenWhitehead merged commit 3a95990 into main Dec 2, 2025
26 checks passed
@BenWhitehead BenWhitehead deleted the build/hermetic-config branch December 2, 2025 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the googleapis/java-storage API. size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants