-
Notifications
You must be signed in to change notification settings - Fork 2
chore(deps): update github actions (major) #71
base: main
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 | ||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -17,7 +17,7 @@ jobs: | |||||||||||||||||||||||||||
| runs-on: ubuntu-latest | ||||||||||||||||||||||||||||
| steps: | ||||||||||||||||||||||||||||
| - name: Checkout | ||||||||||||||||||||||||||||
| uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5 | ||||||||||||||||||||||||||||
| uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6 | ||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||
| - name: Setup Node.js | ||||||||||||||||||||||||||||
| uses: actions/setup-node@2028fbc5c25fe9cf00d9f06a71cc4710d4507903 # v6 | ||||||||||||||||||||||||||||
|
|
@@ -39,7 +39,7 @@ jobs: | |||||||||||||||||||||||||||
| NODE_AUTH_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||
| - name: Upload Distributions | ||||||||||||||||||||||||||||
| uses: actions/upload-artifact@330a01c490aca151604b8cf639adc76d48f6c5d4 # v5 | ||||||||||||||||||||||||||||
| uses: actions/upload-artifact@bbbca2ddaa5d8feaa63e36b76fdaad77386f024f # v7 | ||||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||||
| name: jupyterlab_deepnote-releaser-dist-${{ github.run_number }} | ||||||||||||||||||||||||||||
| path: .jupyter_releaser_checkout/dist | ||||||||||||||||||||||||||||
|
Comment on lines
41
to
45
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. 🧩 Analysis chain🌐 Web query:
💡 Result: In So if you set: - uses: actions/upload-artifact@v7
with:
path: .then anything under include-hidden-files: trueWith Separately (not about dotfiles), Sources: 🏁 Script executed: # Check the workflow file to understand context
cat -n .github/workflows/check-release.yml | sed -n '35,50p'Repository: deepnote/jupyterlab-deepnote Length of output: 584 🌐 Web query:
💡 Result: For Source(s):
Citations: Add
Suggested fix - name: Upload Distributions
uses: actions/upload-artifact@bbbca2ddaa5d8feaa63e36b76fdaad77386f024f # v7
with:
name: jupyterlab_deepnote-releaser-dist-${{ github.run_number }}
path: .jupyter_releaser_checkout/dist
+ include-hidden-files: true
+ if-no-files-found: error📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||
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.
🧩 Analysis chain
🌐 Web query:
actions/download-artifact v8 breaking changes from v6💡 Result:
Breaking changes (and upgrade-impacting changes) when moving from
actions/download-artifact@v6→@v8:@v7runs on Node.js 24 by default (runs.using: node24) and requires Actions Runner ≥ 2.327.1 (notably affects self-hosted runners). In@v6, Node 24 support existed, but the action still defaulted to Node 20. [1] [2]@v8changes the default behavior for a download digest/hash mismatch from warning → error (workflow fails). You can control this with the newdigest-mismatchsetting. [3]@v8no longer blindly unzips everything; it checksContent-Typeand skips unzipping for non-zip downloads (added for “non-zipped artifacts” / direct uploads). There’s a newskip-decompressoption to control this behavior. [3] [4]Sources:
[1]
actions/download-artifactv7.0.0 release notes (Node 24 default + runner minimum)[2]
actions/download-artifactv6.0.0 release notes (Node 24 support, but not default)[3]
actions/download-artifactv8.0.0 release notes (ESM, digest-mismatch default error, decompression changes)[4] GitHub Changelog: non-zipped artifact upload/download support (requires
download-artifact@v8for that mode)Address v6→v8 breaking changes in download-artifact.
v8 has significant breaking changes: artifact decompression behavior changed (now checks Content-Type instead of blindly unzipping), digest mismatches error by default instead of warn, and requires Actions Runner ≥ 2.327.1. Verify workflow doesn't rely on auto-decompression and test digest mismatch handling. Consider adding
digest-mismatchandskip-decompressoptions if needed.🤖 Prompt for AI Agents