-
Notifications
You must be signed in to change notification settings - Fork 3.5k
cai: workflow to auto-fix tests (disabled for now) #7502
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
3 issues found across 2 files
React with 👍 or 👎 to teach cubic. You can also tag @cubic-dev-ai
to give feedback, ask questions, or re-run the review.
permissions: | ||
contents: write | ||
pull-requests: write | ||
issues: write |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Downgrade issues permission to read since the workflow doesn't modify issues.
Prompt for AI agents
Address the following comment on .github/workflows/auto-fix-failed-tests.yml at line 12:
<comment>Downgrade issues permission to read since the workflow doesn't modify issues.</comment>
<file context>
@@ -0,0 +1,170 @@
+permissions:
+ contents: write
+ pull-requests: write
+ issues: write
+ actions: read
+
</file context>
|
||
permissions: | ||
contents: write | ||
pull-requests: write |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reduce pull-requests permission to read to follow least-privilege (no PR write operations are performed).
Prompt for AI agents
Address the following comment on .github/workflows/auto-fix-failed-tests.yml at line 11:
<comment>Reduce pull-requests permission to read to follow least-privilege (no PR write operations are performed).</comment>
<file context>
@@ -0,0 +1,170 @@
+
+permissions:
+ contents: write
+ pull-requests: write
+ issues: write
+ actions: read
</file context>
|
||
- name: Checkout repository | ||
if: steps.workflow-details.outputs.has_failed_tests == 'true' | ||
uses: actions/checkout@v4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Checkout may fail for forked PRs because it doesn't set repository to the PR's head repository; head_sha might not exist in the base repo.
Prompt for AI agents
Address the following comment on .github/workflows/auto-fix-failed-tests.yml at line 92:
<comment>Checkout may fail for forked PRs because it doesn't set repository to the PR's head repository; head_sha might not exist in the base repo.</comment>
<file context>
@@ -0,0 +1,170 @@
+
+ - name: Checkout repository
+ if: steps.workflow-details.outputs.has_failed_tests == 'true'
+ uses: actions/checkout@v4
+ with:
+ token: ${{ secrets.GITHUB_TOKEN }}
</file context>
Description
Disabling this until we have the cubic response workflow running smoothly, but once enabled this will automatically attempt to solve failing tests on PRs
Summary by cubic
Adds a GitHub Actions workflow to auto-fix failing tests on PRs using the Continue CLI. It’s disabled for now while we stabilize the cubic response flow and replaces the older comment-responder workflow.
New Features
Refactors