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

fix: devcontaner.jsonで定義されているpnpmのバージョンが古いため、DevContainerビルド後のパッケージインストールのタイミングでコケる問題を修正 #13819

Closed

Conversation

frfs
Copy link
Contributor

@frfs frfs commented May 10, 2024

What

devcontaner.jsonで定義されているpnpmのバージョンが古いため、DevContainerの立ち上げ時にコケてしまっていました。
このPRでは、package.jsonをもとにDevContainerにインストールするpnpmのバージョンを9.0.6に変更します。

Why

#13818

Additional info (optional)

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@frfs frfs changed the title fix: devcontaner.jsonで定義されているpnpmのバージョンが古いため、DevContainerの立ち上げ時にコケる問題を修正 fix: devcontaner.jsonで定義されているpnpmのバージョンが古いため、DevContainerビルド後のパッケージインストールのタイミングでコケる問題を修正 May 10, 2024
@frfs
Copy link
Contributor Author

frfs commented May 10, 2024

#13814 がマージされるタイミングで9.1.0に揃える必要があるカモ

@zyoshoka
Copy link
Contributor

https://github.com/devcontainers-contrib/features/tree/main/src/corepack を使って Corepack を使うようにする(#10217)とバージョンを一致させる手間がなくなって良さそうです。

@frfs
Copy link
Contributor Author

frfs commented May 11, 2024

#13821 でcorepackを利用した方法のプルリクを立てました。
そちらで問題なければ、このプルリクは不要なものになります。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants