Skip to content

feat: workflow to comment test-models on addition PRs - #856

Merged
LordGameleo merged 5 commits into
mainfrom
harshivg/gateway-2335-automate-test-models-on-pr-title-update-model-yamls
Apr 27, 2026
Merged

feat: workflow to comment test-models on addition PRs#856
LordGameleo merged 5 commits into
mainfrom
harshivg/gateway-2335-automate-test-models-on-pr-title-update-model-yamls

Conversation

@harshiv-26

@harshiv-26 harshiv-26 commented Apr 27, 2026

Copy link
Copy Markdown
Collaborator

Note

Medium Risk
Changes GitHub Actions behavior and introduces a new auto-commenting workflow that runs with PR write/issue permissions and inherited secrets; misconfiguration could spam PRs or unintentionally trigger jobs.

Overview
Adds an automation workflow that, on PR synchronize, posts a /test-models comment when the PR title matches add new models [bot], then runs the model test pipeline using the posted comment id.

Refactors the existing issue_comment-driven test-changed-models workflow to call a new reusable workflow (test-changed-models-impl.yml) that encapsulates the checkout/diff/TrueFoundry trigger logic and makes comment reactions conditional on whether a comment_id was provided.

Reviewed by Cursor Bugbot for commit c8357d3. Bugbot is set up for automated code reviews on this repo. Configure here.

Comment thread .github/workflows/auto-comment-test-models.yml Outdated
Comment thread .github/workflows/auto-comment-test-models.yml Outdated
Comment thread .github/workflows/test-changed-models.yml
Comment thread .github/workflows/auto-test-on-bot-push.yml

@cursor cursor Bot 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.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit c8357d3. Configure here.

Comment thread .github/workflows/auto-test-on-bot-push.yml
@LordGameleo
LordGameleo merged commit 283ad9e into main Apr 27, 2026
5 checks passed
@LordGameleo
LordGameleo deleted the harshivg/gateway-2335-automate-test-models-on-pr-title-update-model-yamls branch April 27, 2026 11:10
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