uhooi / UhooiPicBook

Uhooi's character book.
MIT License
107 stars 25 forks source link

Show warning for long build times #116

Closed tihimsm closed 3 years ago

tihimsm commented 3 years ago

Close #63

configファイル吐き出そうかと思ったけど一旦やめました笑 https://github.com/toshi0383/xcconfig-extractor

uhooi commented 3 years ago

@tihimsm ご対応ありがとうございますー! 修正は project.xml にベタ書きで問題ありません。

ところで相談なのですが、ローカルでビルドしたら型推論に 100 ms 以上かかる箇所が 18 つありましたw

スクリーンショット 2020-10-11 10 57 18

難しいことはやっていないはずなので、警告を 500 ms に伸ばしたいと思うのですが、どうでしょうか? それでよければ直していただけると嬉しいです!(こういうのって私が直すべき?OSSのルールがわかっていない)

uhooi commented 3 years ago

あ、あと 500 ms は私が適当に決めた値なので、他に適切な時間があればご提案していただけると嬉しいです! もちろん他の解決方法(こうすると型推論の時間を短縮できるよ)もあれば大歓迎です!

tihimsm commented 3 years ago

@tihimsm ご対応ありがとうございますー! 修正は project.xml にベタ書きで問題ありません。

ところで相談なのですが、ローカルでビルドしたら型推論に 100 ms 以上かかる箇所が 18 つありましたw

スクリーンショット 2020-10-11 10 57 18

難しいことはやっていないはずなので、警告を 500 ms に伸ばしたいと思うのですが、どうでしょうか? それでよければ直していただけると嬉しいです!(こういうのって私が直すべき?OSSのルールがわかっていない)

レビューありがとうございます! こちらでやりますよー

あ、あと 500 ms は私が適当に決めた値なので、他に適切な時間があればご提案していただけると嬉しいです! もちろん他の解決方法(こうすると型推論の時間を短縮できるよ)もあれば大歓迎です!

秒数は目的に応じてだと思いますが, 普段は500とかでいいかと思います. ビルドに時間がかかることに悩みが出た場合にこの秒数を小さく設定していって細かく見ていくのがいいかなーと!

uhooi commented 3 years ago

ありがとうございます、ではお願いしますー! ですね、今のところビルドが遅いと感じていないので、500msにしておきます!

uhooi commented 3 years ago

LGTMです 👍

uhooi commented 3 years ago

リリースビルド時には不要なことに気づいたので、デバッグビルド時のみフラグを追加するように変更しました! https://github.com/uhooi/UhooiPicBook/commit/59e1d7e4a45ef2e59c7a8e5b2098d7b879e0e50f