Skip to content

fix: Remove paths filter from E2E orchestrator PR trigger to ensure s…#204

Merged
rahuldevikar761 merged 3 commits into
mainfrom
users/radevika/fixWaiting
Feb 2, 2026
Merged

fix: Remove paths filter from E2E orchestrator PR trigger to ensure s…#204
rahuldevikar761 merged 3 commits into
mainfrom
users/radevika/fixWaiting

Conversation

@rahuldevikar761

@rahuldevikar761 rahuldevikar761 commented Feb 2, 2026

Copy link
Copy Markdown
Collaborator

Remove paths filter from E2E orchestrator PR trigger to ensure status always reports

@rahuldevikar761 rahuldevikar761 requested a review from a team as a code owner February 2, 2026 22:05
Copilot AI review requested due to automatic review settings February 2, 2026 22:05

Copilot AI left a comment

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.

Pull request overview

This PR modifies the E2E orchestrator workflow to remove the paths filter from the pull_request trigger, ensuring E2E tests run for all PRs regardless of which files are changed. This prevents the status check from being skipped when changes outside the previously specified paths are made.

Changes:

  • Removed the paths filter from the pull_request trigger in the E2E orchestrator workflow
  • Added a comment explaining that E2E tests should always run to catch regressions

mrunalhirve128
mrunalhirve128 previously approved these changes Feb 2, 2026
abdulanu0
abdulanu0 previously approved these changes Feb 2, 2026
pontemonti
pontemonti previously approved these changes Feb 2, 2026
@rahuldevikar761 rahuldevikar761 enabled auto-merge (squash) February 2, 2026 22:10
@rahuldevikar761 rahuldevikar761 enabled auto-merge (squash) February 2, 2026 22:32
@rahuldevikar761 rahuldevikar761 marked this pull request as draft February 2, 2026 22:33
auto-merge was automatically disabled February 2, 2026 22:33

Pull request was converted to draft

@rahuldevikar761 rahuldevikar761 marked this pull request as ready for review February 2, 2026 22:33
@rahuldevikar761 rahuldevikar761 merged commit e28af6a into main Feb 2, 2026
33 checks passed
@rahuldevikar761 rahuldevikar761 deleted the users/radevika/fixWaiting branch February 2, 2026 22:36
@github-actions

github-actions Bot commented Feb 2, 2026

Copy link
Copy Markdown

✅ All E2E Tests Passed

Sample Status Result
Python OpenAI success
Node.js OpenAI success
Node.js LangChain success
.NET Semantic Kernel success
.NET Agent Framework success

View full test details

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.

5 participants