Skip to content

ci(deserializer): fix npm #451

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.

Sign up for GitHub

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

Merged
TartanLeGrand merged 13 commits into from
Jun 16, 2025
Merged

ci(deserializer): fix npm #451

Changes from 10 commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
7742f52
ci(deserializer): update npm publishing workflow to use new tag input
TartanLeGrand Jun 11, 2025
3dc6a89
ci(deserializer): add build step for SDK in npm publishing workflow
TartanLeGrand Jun 11, 2025
58a4cda
fix: remove tests
TartanLeGrand Jun 12, 2025
2054742
fix: ci artifacts
TartanLeGrand Jun 12, 2025
9d2c0d6
fix: add npm run test:prepare to ci
SeddikBellamine Jun 12, 2025
7ce5af5
fix: use the same workspace
SeddikBellamine Jun 13, 2025
8e12856
fix: build
SeddikBellamine Jun 13, 2025
71c5806
fix: use prepare workspace
SeddikBellamine Jun 13, 2025
430d65e
fix: remove unecessary build steps and use build with the reusable wo…
SeddikBellamine Jun 13, 2025
56dfc22
fix: indentation
SeddikBellamine Jun 13, 2025
a98396a
fix: add set-publish-version job to manage versioning for nightly and…
TartanLeGrand Jun 16, 2025
f6c72d0
fix: update workflow name for clarity in publish-npm-deserializer
TartanLeGrand Jun 16, 2025
ca259d3
fix: update publish-npm workflow reference to version 1.4.0
TartanLeGrand Jun 16, 2025
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
29 changes: 12 additions & 17 deletions .github/workflows/publish-npm-deserializer.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,27 +13,22 @@ on:
default: nightly

jobs:
set-env:
runs-on: ubuntu-latest
outputs:
resolved-environment: ${{ steps.setenv.outputs.env }}
steps:
- id: setenv
run: |
if [ "${{ github.event.inputs.tag }}" = "latest" ]; then
echo "env=production" >> $GITHUB_OUTPUT
else
echo "env=staging" >> $GITHUB_OUTPUT
fi

publish-npm:
needs: set-env
uses: iExecBlockchainComputing/github-actions-workflows/.github/workflows/[email protected]
uses: iExecBlockchainComputing/github-actions-workflows/.github/workflows/publish-npm.yml@feat/publish-npm
with:
scope: '@iexec/deserializer'
registry: 'https://registry.npmjs.org'
node-version: '18'
environment: ${{ needs.set-env.outputs.resolved-environment }}
tag: ${{ github.event.inputs.tag }}
environment: production
working-directory: "packages/dataprotector-deserializer"
install-command: |
npm ci
cd ../sdk
Copy link
Contributor Author

@TartanLeGrand TartanLeGrand Jun 13, 2025

Choose a reason for hiding this comment

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

https://github.com/iExecBlockchainComputing/github-actions-workflows/pull/62/files#diff-f089b0e55b389f533faf1e65e3cb93b0bcd97b391c32f28449d08fd4f0512447R92

have make this for get the artifacts from the other build job

npm ci
npm run codegen
npm run build
cd ../dataprotector-deserializer
npm run test:prepare
Copy link
Contributor Author

@TartanLeGrand TartanLeGrand Jun 13, 2025

Choose a reason for hiding this comment

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

Why we need this ?

secrets:
npm-token: ${{ secrets.NPM_TOKEN }}
npm-token: ${{ secrets.NPM_TOKEN }}
Loading