Skip to content

Change runner version#85

Merged
AHarmlessPyro merged 2 commits into
mainfrom
integration_test_update
May 15, 2026
Merged

Change runner version#85
AHarmlessPyro merged 2 commits into
mainfrom
integration_test_update

Conversation

@AHarmlessPyro
Copy link
Copy Markdown
Contributor

@AHarmlessPyro AHarmlessPyro commented May 15, 2026


Open in Devin Review

Note

Low Risk
CI-only change limited to workflow triggers and runner image; main risk is reduced PR signal or environment differences causing unexpected CI behavior.

Overview
Updates the Integration Tests GitHub Actions workflow to run on ubuntu-24.04 instead of ubuntu-slim.

Removes the pull_request trigger (previously limited to main), leaving the workflow to run only via merge_group and manual workflow_dispatch.

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

Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@AHarmlessPyro AHarmlessPyro merged commit c2673c1 into main May 15, 2026
1 check passed
@AHarmlessPyro AHarmlessPyro deleted the integration_test_update branch May 15, 2026 06:42
Copy link
Copy Markdown

@cursor cursor Bot left a comment

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 64af09d. Configure here.


on:
pull_request:
branches: [main]
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 trigger accidentally removed from CI workflow

High Severity

The pull_request trigger (with branches: [main]) was removed from the on: block, but the PR title only mentions "Change runner version." Integration tests will no longer run on pull requests — only on merge_group checks and workflow_dispatch. This likely means PRs will lose integration test coverage before merging.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 64af09d. Configure here.

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