no4-co-jp / no4-official-site

0 stars 0 forks source link

#4 プロジェクト基盤 #12

Closed asada-no4 closed 1 year ago

asada-no4 commented 1 year ago

close #4

以下を実施 ・astroとsvelteの導入 ・コンポーネントの一部をsvelteコンポーネントに置き換え ・voltaを使ったパッケージマネージャーのバージョン管理

YuutoOse commented 1 year ago

image

asada-no4 commented 1 year ago

@yuki-ohnaka-no4 npmとyarnで比較して、yarnにしました。 pnpmは見たことなかったです。。。 見てみます。

asada-no4 commented 1 year ago

メモ: ・pnpm > yarn > npm の順でインストールとかアプデ速い ・pnpmはライブラリによって、たまにコマンドが対応してない時がある → 手動でセットアップすれば問題ない

runa-sakagoshi0830 commented 1 year ago

image-NGbNRPxsseXhafggSomJ

asada-no4 commented 1 year ago

@yuki-ohnaka-no4

@asada-no4 パッケージマネージャーって、npm, yarn, pnpmあたりが有名所ですが、それぞれ見てみてyarnにした感じですか?👀

https://github.com/no4-co-jp/no4-official-site/pull/12#issuecomment-1429027553

調べたところこんな感じだったのでpnpmでも良かったかなと思ってます。 pnpmを使った修正コミットを積んでおきます。

yuki-ohnaka-no4 commented 1 year ago

@asada-no4 いじめるわけじゃないですが、、、 選定観点の懸念として...

asada-no4 commented 1 year ago

@yuki-ohnaka-no4 npmだとバージョンが統一されないっていうのは見たんですが、そっちのほうが都合が良いですかね? 知識がないなりに、開発者によってバグが発生したりしなかったり、っていうのがあると怖いのかなと思ってました。

yuki-ohnaka-no4 commented 1 year ago

@asada-no4 npm ci ちょっと調べてみるといいかも👀

asada-no4 commented 1 year ago

メモ:

https://qiita.com/mstssk/items/8759c71f328cab802670 npm ci

asada-no4 commented 1 year ago

パッケージマネージャーをnpmに変更しました

yuki-ohnaka-no4 commented 1 year ago

@asada-no4 npmにするとして、npmのバージョン管理はどうしましょう?👀

asada-no4 commented 1 year ago

@yuki-ohnaka-no4 voltaが気になっているので、ちょっと見てみます

asada-no4 commented 1 year ago

voltaを使ってバージョンを指定してみました

yuki-ohnaka-no4 commented 1 year ago

@asada-no4 最後にパッケージ選定のまとめをざっと書いてくれるとうれしい👀

asada-no4 commented 1 year ago

パッケージ選定の流れ

yarn を使用 → npmよりも高速であることと、依存プログラム/パッケージのバージョン管理に優れていることから → pnpmがより高速でpackage.json と package-lock.json の整合性をチェックしてくれるためそちらを検討 → pnpmはパッケージのグローバル管理で懸念があるし、利用不可のライブラリあり & yarnはバージョンアップで破壊的変更 → npmのciコマンドを使えばバージョンが違いのエラー出してくれる & スタートアップのときの速さはそんなにいらない → npmを使う バージョン管理はvoltaを使うことでカバー

yuki-ohnaka-no4 commented 1 year ago

@asada-no4

npmのciコマンドを使えばバージョンが違いのエラー出してくれる & スタートアップのときの速さはそんなにいらない

https://blog.minimalcorp.com/users/jigen/posts/6f325dc9b8a00370b6aedf47a34cb3ce

npm ciのちょっと利点がずれてる気がするので、これ軽く読んでおいてくださいなー

yuki-ohnaka-no4 commented 1 year ago

@asada-no4 全員approveなので、マージお願いします〜