forked from scs-sandbox/VAmPI
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Review-copilot action tests #4
Merged
Merged
Changes from all commits
Commits
Show all changes
16 commits
Select commit
Hold shift + click to select a range
e6ce575
Review-copilot action tests
nierz cf1cffc
Update review-copilot.yml
nierz 9ad3f7c
Update review-copilot.yml
nierz 63d5c85
Update review-copilot.yml
nierz 40692e5
Update review-copilot.yml
nierz de12a22
Update review-copilot.yml
nierz da7a2c9
Update review-copilot.yml
nierz c981f9e
Update review-copilot.yml
nierz 2e1f011
Update review-copilot.yml
nierz aa20c06
Update review-copilot.yml
nierz 19b753c
Update review-copilot.yml
nierz ef18740
Update review-copilot.yml
nierz 3aaedfe
Update review-copilot.yml
nierz e8a073b
Update review-copilot.yml
nierz 213fae8
Update review-copilot.yml
nierz c25da8d
Update review-copilot.yml
nierz File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,44 @@ | ||
name: Review Co-Pilot | ||
|
||
on: | ||
pull_request: | ||
types: [reopened, opened, synchronize] | ||
|
||
jobs: | ||
review-copilot: | ||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- name: Checkout code | ||
uses: actions/checkout@v3 | ||
|
||
- name: Set up Python | ||
uses: actions/setup-python@v4 | ||
with: | ||
python-version: '3.x' | ||
|
||
- name: Install GitHub API library | ||
run: | | ||
python -m pip install --upgrade pip | ||
pip install PyGithub | ||
|
||
- name: Install pipx | ||
run: | | ||
python -m pip install --upgrade pip | ||
python -m pip install pipx | ||
|
||
- name: Install Review Copilot | ||
run: | | ||
pipx install "git+https://${{ secrets.REVIEW_COPILOT_KEY }}@github.com/scs-sandbox/review-copilot.git" | ||
|
||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Missing error handling for the review-copilot command execution. The workflow should check the exit code and fail explicitly if the review fails. |
||
- name: Run Review Copilot | ||
env: | ||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
AWS_ACCESS_KEY_ID: ${{ secrets.REVIEW_COPILOT_ACCESS_KEY }} | ||
AWS_SECRET_ACCESS_KEY: ${{ secrets.REVIEW_COPILOT_SECRET_ACCESS_KEY }} | ||
|
||
run: | | ||
REPO_NAME="${{ github.repository }}" | ||
PR_NUMBER="${{ github.event.pull_request.number }}" | ||
|
||
review-copilot review $REPO_NAME $PR_NUMBER |
Oops, something went wrong.
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.
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.
Consider specifying a more precise Python version instead of '3.x' to ensure consistent execution environment.