-
Notifications
You must be signed in to change notification settings - Fork 0
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
Use v 9.1.0 of pnpm #826
Use v 9.1.0 of pnpm #826
Conversation
|
✅ Deploy Preview for obos-grunnmuren ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@@ -46,7 +46,7 @@ | |||
"tailwindcss": "3.4.3", | |||
"typescript": "5.4.5" | |||
}, | |||
"packageManager": "pnpm@9.0.6", | |||
"packageManager": "pnpm@9.1.0", |
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.
Er på telefonen nå, så ikke helt sikker. Men er det ikke noe greier vi kan sette i pnpm configen som sier at denne ikke er strict?
Eventuelt kan verdien i package.json endres til å ikke være strict? Om jeg er på versjon 9.1.0 eller 9.1.1 er ikke så interessant tenker jeg. Det viktigste er at man ikke bruker pnpm 8!
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.
Ja, jeg var litt inne på samme tanke. Er litt slitsomt å måtte oppdatere den for hver enste minor og patch. Man kan vel ofte skrive ranges i package.json
, men ser ikke ut som det støttes enda: nodejs/corepack#208 (markert som duplikat av nodejs/corepack#95, som fortsatt er åpen)
Oppgraderer til pnpm v9.1.0