-
Notifications
You must be signed in to change notification settings - Fork 12
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
🔧 Setup GitHub actions #3374
base: v2-bridge
Are you sure you want to change the base?
🔧 Setup GitHub actions #3374
Changes from 3 commits
1f4d69b
36af443
67d3481
1196fb1
8da7309
459633f
a2232b9
b3b85f6
2ce1b1d
7728cb4
b109386
d6bad9d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Original file line number | Diff line number | Diff line change | ||||||||
---|---|---|---|---|---|---|---|---|---|---|
@@ -0,0 +1,38 @@ | ||||||||||
# This workflow will run tests using node and then publish a package to GitHub Packages when a release is created | ||||||||||
# For more information see: https://docs.github.com/en/actions/publishing-packages/publishing-nodejs-packages | ||||||||||
|
||||||||||
name: Node.js Package | ||||||||||
deraw marked this conversation as resolved.
Show resolved
Hide resolved
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.
Suggested change
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. c etait pour avoir les sources qui m ont aidé à le mettre en place 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. Il faut mettre à jour le commentaire alors (ou ne laisser que le lien) |
||||||||||
|
||||||||||
on: | ||||||||||
release: | ||||||||||
types: [created] | ||||||||||
|
||||||||||
jobs: | ||||||||||
build: | ||||||||||
runs-on: ubuntu-latest | ||||||||||
steps: | ||||||||||
- name: Checkout repository | ||||||||||
uses: actions/checkout@v4 | ||||||||||
- uses: pnpm/action-setup@v2 | ||||||||||
with: | ||||||||||
version: 8 | ||||||||||
|
||||||||||
- uses: actions/setup-node@v3 | ||||||||||
with: | ||||||||||
node-version: 20.11.0 | ||||||||||
deraw marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||||||
# cache: 'pnpm' | ||||||||||
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.
Suggested change
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. en local c est l enfer pour tester mais on peut le mettre en prod 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. Il faudrait le variabiliser du coup non ? |
||||||||||
- run: corepack enable | ||||||||||
- run: pnpm install --frozen-lockfile | ||||||||||
|
||||||||||
publish-npm: | ||||||||||
needs: build | ||||||||||
runs-on: ubuntu-latest | ||||||||||
steps: | ||||||||||
- uses: actions/checkout@v3 | ||||||||||
- uses: actions/setup-node@v3 | ||||||||||
with: | ||||||||||
node-version: 20.11.0 | ||||||||||
registry-url: https://registry.npmjs.org/ | ||||||||||
- run: npm publish | ||||||||||
env: | ||||||||||
NODE_AUTH_TOKEN: ${{secrets.npm_token}} |
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.
Il faut laisser ce fichier pour que CircleCI ignore cette branche et celles qui sont crées à partir de celle-ci
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.
je l ai remis
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.
mais j ai desacitver le circle ci , donc je suis pas sur que cela soit necessaire ?? non ?
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.
Il faut le mettre vu qu'on a réactivé CircleCI pour la v2