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

Feil: Linker-scriptet fungerer ikke for pnpm #3309

Closed
wkillerud opened this issue Dec 6, 2022 · 2 comments · Fixed by #3315
Closed

Feil: Linker-scriptet fungerer ikke for pnpm #3309

wkillerud opened this issue Dec 6, 2022 · 2 comments · Fixed by #3315
Assignees
Labels
🐝 bug Noe er feil eller ikke som det skal 👷‍♂️ CI & infra Automatisering, bygging, publisering og infrastruktur

Comments

@wkillerud
Copy link
Contributor

Linker-scriptet legger opp til at pnpm fungerer på samme måte som yarn og npm med en tostegs setup. Det stemmer ikke.

npm linker i to steg:

  1. npm link kjøres i mappa til pakka som skal linkes til
  2. npm link <pakkenavn kjøres i mappa til prosjektet som bruker pakka.

pnpm gjør bare det sistnevnte, og da basert på filstien til pakka som skal linkes til, ikke pakkenavn.

Steg for å gjenskape

  1. Kjør pnpm linker fra Jøkul og pek på et prosjekt som bruker pnpm (har en pnpm-lock.yaml)

Forventet oppførsel

Alle Jøkul-pakker skal linkes opp.

@wkillerud wkillerud added 🐝 bug Noe er feil eller ikke som det skal 👷‍♂️ CI & infra Automatisering, bygging, publisering og infrastruktur labels Dec 6, 2022
@wkillerud wkillerud self-assigned this Dec 7, 2022
wkillerud added a commit that referenced this issue Dec 8, 2022
@wkillerud
Copy link
Contributor Author

Har en fix for pnpm, men oppdager at linkeren ikke fungerer lenger for npm etter at vi byttet package manager. Det er en kjent sak. Forstår at npm ikke kan forventes å forstå node_modules-strukturen til alle andre package managers under solen.

En workaround er at folk installerer pnpm og gjør en "midlertidig migrering":

  • rm package-lock.json && rm -rf node_modules
  • pnpm install

Men er ikke sikkert det er så enkelt å migrere. Vi ble jo nødt til å konfigurere om Webpack blant annet. Så denne er litt bummer, rett og slett.

Har ikke testet andre veien, om en pnpm link til en pakke i et npm workspace fungerer.

Jeg er fristet til å si at det å ikke kunne linke opp Jøkul med standard package manager i økosystemet er not good, og taler til fordel for å droppe pnpm til fordel for vanilla npm.

Noe å ta opp på frontendforum kanskje @piofinn?

@wkillerud
Copy link
Contributor Author

Har ikke testet andre veien, om en pnpm link til en pakke i et npm workspace fungerer.

@espkva @hjellek, har dere testet dette før? Tanker om dilemmaet jeg beskriver over?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐝 bug Noe er feil eller ikke som det skal 👷‍♂️ CI & infra Automatisering, bygging, publisering og infrastruktur
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant