-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
feat(workflows): migrate pr-test to rari #25577
Conversation
@@ -42,12 +58,14 @@ | |||
key: dotlocal-${{ runner.os }}-${{ steps.setup-python.outputs.python-version }}-0 | |||
|
|||
- name: Install Python poetry | |||
uses: snok/install-poetry@v1 | |||
if: env.HAS_ARTIFACT | |||
uses: snok/[email protected] |
Check warning
Code scanning / CodeQL
Unpinned tag for a non-immutable Action in workflow Medium
Uses Step
Preview URLs Flaws (1)URL:
(comment last updated: 2025-01-24 12:52:17) |
f60d9eb
to
6e450c2
Compare
The existing `pr-test` workflow was renamed to `pr-test-legacy`. Both workflows now use the `pull_request` event, while `pr-review-companion` uses the `workflow_run` event. # Conflicts: # .github/workflows/pr-review-companion.yml # .github/workflows/pr-test.yml
6e450c2
to
145be21
Compare
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.
lgtm
Description
Migrates the
pr-test
workflow to rari.Motivation
Additional details
The existing
pr-test
workflow was renamed topr-test-legacy
, and disabled.Both workflows now use the
pull_request
event, whilepr-review-companion
uses theworkflow_run
event.Related issues and pull requests
Related content PRs: