Skip to content
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

[TECH] Aligner les montées de versions npm / node sur le mono repo (PIX-6533) #5371

Merged
merged 5 commits into from
Dec 16, 2022

Conversation

xav-car
Copy link
Contributor

@xav-car xav-car commented Dec 14, 2022

🎄 Problème

On ne veut pas imposer une version mineure de node, mais pix-ui en imposait une (transitivement).

🎁 Proposition

Suite à cet ADR.

Monter en version 20.2.4 de pix-ui qui ne vérifie plus que la version majeure de node.
Puis relâcher le check dans le package.json et npm install pour mettre à jour le package-lock.json.

Impact : Notre version de prod de node devient la 16.19.0, et npm 8.19.3.

🌟 Remarques

J'ai remplacé toutes nos références à [email protected] vers 16.18.1.

🎅 Pour tester

Vérifier que tout démarre sur les RA.

Vérifier que le script de configure.sh à l'onboarding fonctionne toujours.

@xav-car xav-car added 👀 Tech Review Needed cross-team Toutes les équipes de dev labels Dec 14, 2022
@xav-car xav-car self-assigned this Dec 14, 2022
@xav-car xav-car force-pushed the pix-6533-relax-npm-package branch 2 times, most recently from a279bf1 to 79881aa Compare December 16, 2022 09:46
@octo-topi octo-topi changed the title [TECH] Aligner les monter de versions npm / node sur le mono repo(PIX-6533) [TECH] Aligner les monter de versions npm / node sur le mono repo (PIX-6533) Dec 16, 2022
@yannbertrand yannbertrand self-assigned this Dec 16, 2022
@pix-service
Copy link
Contributor

@octo-topi octo-topi changed the title [TECH] Aligner les monter de versions npm / node sur le mono repo (PIX-6533) [TECH] Aligner les montées de versions npm / node sur le mono repo (PIX-6533) Dec 16, 2022
@yannbertrand yannbertrand added the team-captains This is your captain speaking label Dec 16, 2022
@yannbertrand yannbertrand force-pushed the pix-6533-relax-npm-package branch from 5ca61b6 to 5ead095 Compare December 16, 2022 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-team Toutes les équipes de dev 🚀 Ready to Merge team-captains This is your captain speaking Tech Review OK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants