Skip to content

Improve: mise コマンドの追加#504

Merged
den0206 merged 2 commits into
mainfrom
feature/GH-499
May 27, 2025
Merged

Improve: mise コマンドの追加#504
den0206 merged 2 commits into
mainfrom
feature/GH-499

Conversation

@den0206

@den0206 den0206 commented May 27, 2025

Copy link
Copy Markdown
Contributor

概要


  • mise run <コマンド名> で実行可能なコマンドを追加しました

拡張機能を入れておくとボタンでも実行できます
スクリーンショット 2025-05-27 15 59 48

レビュー観点

  • mise run clean が実行できるか?
  • mise run cspell が実行できるか?
  • 他に追加が必要なコマンドはないか?

レビューレベル

  • Lv1: ぱっとみて違和感がないかチェックして Approve する
  • Lv2: 仕様レベルまで理解して、仕様通りに動くかある程度検証して Approve する
  • Lv3: 実際に環境で動作確認したうえで Approve する

レビュー優先度

  • すぐに見てもらいたい ( hotfix など ) 🚀
  • 今日中に見てもらいたい 🚗
  • 今日〜明日中で見てもらいたい 🚶
  • 数日以内で見てもらいたい 🐢

画像 / 動画

Before After Design

確認したこと

  • コマンドの実行

動作確認手順

  • mise run clean の実行
  • mise run cspell の実行

備考

@yumemi-team-review-requester yumemi-team-review-requester Bot requested review from a team, macneko-ayu and masa-futa and removed request for a team May 27, 2025 07:07
@github-actions

Copy link
Copy Markdown

Ready for review 🚀

Comment thread mise.toml
Comment on lines +18 to +22
[tasks.cspell]
description = "Run cspell to check spelling."
run = [
'bun run cspell lint',
]

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

imo-badge

bun run cspell lint 自体を忘れそうなので追加しました
(mise.toml に書いてあれば忘れずに使用ができる)
不必要であれば削除します

@blendthink blendthink left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

とてもいいと思いました 🫶
LGTM です!
ご対応ありがとうございます!

Comment thread package.json
"husky": "^9.1.7"
},
"scripts": {
"clean": "git clean -xdf node_modules",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

good-badge
🫶

@den0206 den0206 added this pull request to the merge queue May 27, 2025
Merged via the queue into main with commit e1835f9 May 27, 2025
13 checks passed
@den0206 den0206 deleted the feature/GH-499 branch May 27, 2025 08:47
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.

[Improve]: mise コマンドの追加

2 participants