yumemi-inc / flutter-mobile-project-template

MIT License
37 stars 7 forks source link

GitHub Actionsのワークフローで 適切なmelosバージョンを利用するよう修正 #161

Closed YumNumm closed 8 months ago

YumNumm commented 8 months ago

[!NOTE]

  • #156 のmainへのマージ待ち (#156のベースブランチから、本ベースブランチを切っています)
    • 既にマージ済み

概要

レビュー観点

レビューレベル

レビュー優先度

メモ

github-actions[bot] commented 8 months ago

Ready for review :rocket:

github-actions[bot] commented 8 months ago

Visit the preview URL for this PR (updated for commit eaec5e6):

https://flutter-mobile-project-template-catalog--pr161-feature-hi4n2oqc.web.app

(expires Tue, 19 Mar 2024 10:16:39 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 9ea56735a63d07a7cfe62eb204b0528284c37c23

blendthink commented 8 months ago

@YumNumm 完全に見落としていたのですが、check-diff.yamlcheck-pr.yamlmelos run の部分を dart run melos run に修正する必要がありそうです 🙏

YumNumm commented 8 months ago

@blendthink Slackにて話した件、対応させていただきました。 お時間ある時にレビューよろしくお願いします。

(以下、Slackより引用)

環境変数PATH にmelosが無いとmelosが正常に動かなさそうです 以下の対処で考えているのですが、いかがでしょうか?

現象

  • dart run melos run hogehoge でmelos.yaml に定義したスクリプトを実行すると、melosはmelos自身を実行しようとする
    • 環境変数PATH にmelosが無いため実行できない (画像1)
    • CI(GitHub Actions)上でも同様の現象が起きる

      対処

      dart run 経由でmelosを実行するのは厳しそう →

  • pubspec.lock からmelosのバージョンを取得し dart pub global activate melos x.y.z する
    • melosバージョン取得はyq コマンドでできそうです(画像2)
Pic1 Pic2
image image
github-actions[bot] commented 8 months ago

Ready for review :rocket:

YumNumm commented 8 months ago

memo-badge

image