Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions .github/workflows/cicd_npm-publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,11 @@ jobs:
publish:
if: ${{ github.event.label.name == 'npm-ready-for-publish' }}
uses: pipedrive-actions/github-actions-workflows/.github/workflows/reusable_cicd-npm-publish.yml@master
permissions:
id-token: write
contents: write
with:
revision: ${{ github.event.pull_request.head.ref }}
supported_versions: '[22]'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In .nvmrc the Node version was set to 24. Is it correct that here it stays unchanged? Also, in package.json the version is still set to >=22 (yes, the 24 is covered :), but would it make sense to change it there as well?)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With this setup, the publish workflow runs the tests with version 24 (main version based on the .nvmrc) and runs with version 22, which respects the package.json

use_trusted_publisher: true
secrets: inherit
2 changes: 1 addition & 1 deletion .nvmrc
Original file line number Diff line number Diff line change
@@ -1 +1 @@
22
24
Loading