Fix workflows to use stable @main action reference#22
Open
devin-ai-integration[bot] wants to merge 4 commits into
Open
Fix workflows to use stable @main action reference#22devin-ai-integration[bot] wants to merge 4 commits into
devin-ai-integration[bot] wants to merge 4 commits into
Conversation
- Update devin-code-review.yml to use @main instead of PR branch - Update devin-test-fix.yml to use @main instead of PR branch - Now that the Devin API action is merged to main, workflows should reference the stable version - This fixes the 'Unable to resolve action' CI error Related to: - DEVIN-GITHUB-ACTIONS PR #1: samfert/DEVIN-GITHUB-ACTIONS#1 - BankingMicroservices PR #21: #21 - Devin session: https://app.devin.ai/sessions/8fd0bbe89bdf48e79b4caa52bb28ad82 - Requested by: @samfert-codeium Co-Authored-By: Sam Fertig <sam.fertig@codeium.com>
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
5 tasks
The DEVIN-GITHUB-ACTIONS repository has been made public, which resolves the 'Unable to resolve action' error. This commit triggers a new CI run to verify the action can now be resolved correctly. Co-Authored-By: Sam Fertig <sam.fertig@codeium.com>
The DEVIN-GITHUB-ACTIONS repository now has the JSON escaping fix merged to main. This empty commit triggers CI to re-run and pick up the fixed action from @main reference. Co-Authored-By: Sam Fertig <sam.fertig@codeium.com>
Co-Authored-By: Sam Fertig <sam.fertig@codeium.com>
🤖 Devin Code ReviewDevin is reviewing this pull request for:
Session URL: https://app.devin.ai/sessions/e9c44bdcd07d4832b0a56f591047172c Devin will analyze the changes and provide feedback. You can monitor the progress at the session URL above. This review was automatically triggered by the GitHub Actions workflow. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fix workflows to use stable @main action reference
Summary
Updates the Devin API action references in two workflows (
devin-code-review.ymlanddevin-test-fix.yml) to use the stable@mainbranch instead of the temporary PR branch@devin/1762304734-devin-action-implementation.Changes:
samfert-codeium/DEVIN-GITHUB-ACTIONS/devin-action@mainContext: PRs #1 (DEVIN-GITHUB-ACTIONS) and #21 (BankingMicroservices) were merged with temporary PR branch references. Now that the Devin API action is merged to main, these workflows need to reference the stable version.
Review & Testing Checklist for Human
DEVIN_API_KEYTest Plan
Notes
Why This PR Exists: The original PRs (#1 and #21) were merged for testing purposes before the action repository was made public. They contained references to the PR branch (
@devin/1762304734-devin-action-implementation) which worked temporarily but should be updated to the stable@mainreference now that the action is merged.Known Issue: CI is currently failing with "Unable to resolve action
samfert-codeium/devin-github-actions, not found" because the repository is private. This is NOT a bug in this PR - it's a repository visibility configuration issue that requires making the DEVIN-GITHUB-ACTIONS repository public.What Happens After Making Public: The workflow may still fail if
DEVIN_API_KEYsecret is missing, but this is expected behavior and documented in the workflow README.Related PRs:
Session Link: https://app.devin.ai/sessions/8fd0bbe89bdf48e79b4caa52bb28ad82
Requested by: @samfert-codeium (Sam Fertig)