Skip to content

ci: require both checks on main pull requests#2

Merged
GopiB9119 merged 2 commits into
mainfrom
chore/ci-main-pr-checks
Mar 5, 2026
Merged

ci: require both checks on main pull requests#2
GopiB9119 merged 2 commits into
mainfrom
chore/ci-main-pr-checks

Conversation

@GopiB9119

Copy link
Copy Markdown
Owner

Summary

  • Ensure python-agent-tests always runs for PRs to main (remove path-filter skips).
  • Scope eval-gate to PRs targeting main.
  • Add workflow-level concurrency to cancel superseded runs.

Why

Branch protection requires test-python-agent and evaluate checks. Path-filter skips could leave required checks missing on some PRs.

Validation

  • npm run agent:test:py (25 tests passed)

Copilot AI review requested due to automatic review settings March 5, 2026 09:53

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR ensures that both required branch-protection checks (test-python-agent and evaluate) always run on PRs targeting main, by removing path-based filters that could cause them to be skipped. It also adds workflow-level concurrency groups to cancel superseded runs.

Changes:

  • Remove path filters from python-agent-tests.yml and add branches: [main] filters for both push and pull_request triggers, ensuring tests always run.
  • Scope eval-gate.yml's pull_request trigger to the main branch only.
  • Add concurrency blocks with cancel-in-progress: true to both workflows.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/python-agent-tests.yml Replaced path filters with branch filters (main); added concurrency group to cancel superseded runs.
.github/workflows/eval-gate.yml Scoped pull_request to main branch; added concurrency group to cancel superseded runs.

Comment thread .github/workflows/python-agent-tests.yml
@GopiB9119 GopiB9119 merged commit 40fe516 into main Mar 5, 2026
2 checks passed
@GopiB9119 GopiB9119 deleted the chore/ci-main-pr-checks branch March 5, 2026 10:26
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