Skip to content

build: mise によるバージョン管理を追加 #19

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

Merged
merged 2 commits into from
Apr 19, 2025

Conversation

tatsutakein
Copy link
Contributor

@tatsutakein tatsutakein commented Apr 19, 2025

Issue

概要

mise によるバージョン管理を追加します。

詳細

  • mise ( asdf 互換あり) で bun, flutter, nodejs のバージョン管理を開始
  • bun.lock, package.json を新規作成し、[email protected] を devDependencies に追加
  • package.json に clean スクリプトを追加(git clean -xdf node_modules

プロジェクト参画時の手順

mise が導入してある前提で記載します。

# mise による bun, flutter, nodejs のインストール
mise install

# bun による supabase cli のインストール
bun install

対応していないこと

画像・動画

なし

その他

@tatsutakein tatsutakein marked this pull request as ready for review April 19, 2025 00:15
@tatsutakein tatsutakein enabled auto-merge April 19, 2025 00:15
@tatsutakein tatsutakein self-assigned this Apr 19, 2025
@@ -0,0 +1,3 @@
bun 1.2.10
flutter 3.29.3-stable
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

memo-badge

Flutter は現状の最新バージョンを指定しています。

"clean": "git clean -xdf node_modules"
},
"devDependencies": {
"supabase": "2.20.12"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

memo-badge

mise (asdf プラグイン) を利用することも検討しましたが JS 系の cli なので素直にこっちで管理するほうが良さそうに思いこちらに記載しています。

Copy link
Collaborator

@blendthink blendthink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tatsutakein
少しだけコメントさせていただきましたのでご確認お願いいたします 🙏

@@ -0,0 +1,3 @@
bun 1.2.10
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

imo-badge
@YumNumm @a-skua
pnpm もしくはその他のツールを利用するかをはじめに議論していた方が良いかなと思いますので、ご確認お願いします 🙏

個人的には bun でいいかなと思いつつ、昨年と同様にモノレポでいろんなパッケージを使うのであれば pnpm の方がいいのかもしれないと思いました。

memo-badge
https://qiita.com/syukan3/items/0e3560bf0e2ae715f5d1#3-%E4%B8%BB%E8%A6%81%E3%81%AA%E6%AF%94%E8%BC%83%E3%83%9D%E3%82%A4%E3%83%B3%E3%83%88

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

js周り、パッケージマネージャーが乱立してるので、個人的にnode使うなら純粋にnpmで良いのではと思ってたりしますけれども、そんなにこだわりないのでbunでも良いと思います〜

(あえて何か言うことがあるとすれば、Deno fmt使おうかなと思ってたくらいです)

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@YumNumm
メンション投げてしまってますが、いったんウェブサイトチームとしては bun 使う方針で OK ということで、このまま進めさせていただきます🙏

もし何か懸念点などあればまた定例の際にご相談いただければと思います!

Copy link
Collaborator

@blendthink blendthink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ご対応ありがとうございました💡
LGTM です!

@tatsutakein tatsutakein merged commit 0806b6e into main Apr 19, 2025
@tatsutakein tatsutakein deleted the tatsutakein/GH-18 branch April 19, 2025 14:13
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.

[All] mise によるバージョン管理を追加
3 participants