Skip to content

mise run appを修正#595

Open
hikaru-0602 wants to merge 7 commits into
mainfrom
fix/build-error
Open

mise run appを修正#595
hikaru-0602 wants to merge 7 commits into
mainfrom
fix/build-error

Conversation

@hikaru-0602
Copy link
Copy Markdown
Contributor

@hikaru-0602 hikaru-0602 commented May 26, 2026

案件: [案件名](Notion URL)

やったこと

  • mise run app に外部ライブラリダウンロード処理追加
  • Gemのバージョンに差分合ったのでついでにCommit

確認したこと

  • ビルドできる事を確認

Comment thread mise.toml
run = [
"melos run gen:build_runner --no-select",
"cd apps/dotto && flutter run --dart-define-from-file=./dart_defines/development.json",
"cd apps/dotto/ios && xcodebuild -resolvePackageDependencies -workspace Runner.xcworkspace -scheme Runner",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

うーん、これ必要かなぁ

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.

俺もあぼちゃんもビルドできなくて,このコマンド追加したらできました

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

なんでこれが必要かも調査してもらえません...??
mise setup か mise run app のどこかのタイミングで実行されるflutter or melosコマンドのどこかのタイミングで勝手にSPMのResolveが走ると思うから、必要ないと思うんだけど...

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

少なくとも、僕と @masaya-osuga はこれがなくてもできたから

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.

よくわからないけどキャッシュが原因だそうです
みふゆさんも最初はエラー起きてたけどこれでビルド出来てました

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

来週帰ったら調査します

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@hikaru-0602
DerivedDataの削除とか試した?

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.

やってないです

Comment thread Gemfile.lock
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

mise run app 実行時に iOS 側の外部依存(Swift Package)の解決を先に行うことで、初回起動や依存更新時のビルド失敗を減らすことを目的としたPRです。あわせて Bundler のロックファイルにプラットフォーム差分が入っています。

Changes:

  • mise run appxcodebuild -resolvePackageDependencies を追加し、iOS 依存解決を事前実行するよう変更
  • flutter run の実行前に cd を分割(ただし現状の相対パスに不整合あり)
  • Gemfile.lockarm64-darwin-25 プラットフォームを追加

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

File Description
mise.toml mise run app に iOS の依存解決ステップを追加し、起動手順を調整
Gemfile.lock Bundler の対応プラットフォームに arm64-darwin-25 を追加

Comment thread mise.toml Outdated
Comment thread Gemfile.lock Outdated
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 26, 2026

1 Error
🚫 PRにMilestoneが設定されていません。適切なMilestoneを設定してください。
1 Message
📖 Dangerのチェックが完了しました! 🎉

Generated by 🚫 Danger

hikaru-0602 and others added 4 commits May 26, 2026 18:07
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.

4 participants