-
Notifications
You must be signed in to change notification settings - Fork 645
chore(deps): update github workflows #6733
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: master
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -19,17 +19,17 @@ | |
|
|
||
| steps: | ||
| - name: Setup Python | ||
| uses: actions/setup-python@a309ff8b426b58ec0e2a45f0f869d46889d02405 # v6 | ||
| uses: actions/setup-python@5fda3b95a4ea91299a34e894583c3862153e4b97 # v7 | ||
| with: | ||
| python-version: 3.14t | ||
|
|
||
| - name: Setup Node.js | ||
| uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6.4.0 | ||
| uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0 | ||
| with: | ||
|
Check warning on line 28 in .github/workflows/ai-integration-test.yml
|
||
|
Contributor
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. Node.js version 24 does not exist and will break workflow
Evidence
Identified by Warden · find-bugs · E4V-FZT |
||
| node-version: '20' | ||
| node-version: '24' | ||
|
|
||
| - name: Checkout repo | ||
| uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6.0.3 | ||
| uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1 | ||
| with: | ||
| token: ${{ secrets.GITHUB_TOKEN }} | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -21,7 +21,7 @@ | |
| os: [ubuntu-22.04] | ||
| services: | ||
| postgres: | ||
| image: ghcr.io/getsentry/image-mirror-library-postgres:17-alpine | ||
| image: ghcr.io/getsentry/image-mirror-library-postgres:18-alpine | ||
|
Check failure on line 24 in .github/workflows/test-integrations-dbs.yml
|
||
|
Contributor
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. PostgreSQL 18 image does not exist and will break DB test jobs The postgres service was bumped to Evidence
Identified by Warden · code-review · PDN-ZQ7 |
||
| env: | ||
| POSTGRES_PASSWORD: sentry | ||
| # Set health checks to wait until postgres has started | ||
|
|
@@ -54,9 +54,9 @@ | |
| SENTRY_PYTHON_TEST_MYSQL_PASSWORD: sentry | ||
| container: ${{ (matrix.python-version == '3.6' || matrix.python-version == '3.7') && format('python:{0}', matrix.python-version) || null }} | ||
| steps: | ||
| - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6.0.3 | ||
| - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1 | ||
| - name: Install uv | ||
| uses: astral-sh/setup-uv@fac544c07dec837d0ccb6301d7b5580bf5edae39 # v8.2.0 | ||
| uses: astral-sh/setup-uv@c771a70e6277c0a99b617c7a806ffedaca235ff9 # v9.0.0 | ||
| with: | ||
| enable-cache: false | ||
| - name: "Setup ClickHouse Server" | ||
|
|
||
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.
Invalid setup-python action pin
High Severity
actions/setup-pythonis pinned to5fda3b95...with comment# v7, but the PR’s own update table targets digestece7cb0(publishedv6.3.0), and there is no publishedsetup-pythonv7release matching that SHA. The AI integration workflow can fail at action resolution before tests run.Reviewed by Cursor Bugbot for commit ae76386. Configure here.