chore: Remove push triggers for CI
#3474
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Which issue does this PR close?
Closes #.
Rationale for this change
When new PR is created it might start duplicated CI checks like in #3470
What changes are included in this PR?
Remove duplicate CI runs by disabling push triggers on PR workflows
Problem
Currently, workflows with both
pushandpull_requesttriggers run twice:pushevent)pull_requestevent)This causes:
Solution
Comment out
pushtriggers in PR-focused workflows, keeping onlypull_request:pr_build_linux.ymlpr_build_macos.ymliceberg_spark_test.ymlspark_sql_test.ymlpr_benchmark_check.ymlmiri.ymlvalidate_workflows.ymlWhy pull_request is sufficient
The
pull_requesttrigger runs when:This covers all the validation needed before merging. Direct pushes to feature branches don't need separate CI runs since they'll be tested when the PR is created/updated.
Manual override still available
All workflows retain
workflow_dispatchfor manual triggering when needed.How are these changes tested?