Closed Hiroshiba closed 1 year ago
@qryxip 正直ちょっと若干自信がないのですが、少なくとも今回の場合は大丈夫そうです!
というのも今回はinputs.ほげ
を'true'
と比較しているので、中身が文字列であろうとbooleanであろうと文字列にキャストされて比較されると思います。
https://docs.github.com/ja/actions/learn-github-actions/expressions#contains
== 'true'
を使っている限りではどっちでも安全なはず。(これが一番信頼できるはず)
ちなみにinputs(正式にはinputsコンテキスト)はgithub.event.inputsを又渡ししているだけだと思うのですが、github.event.inputs
でstringだったboolean値がinputs
でちゃんとしたbooleanになるのかはドキュメントに書いてませんでした・・・。
レビューありがとうございます!マージします!
というのも今回は
inputs.ほげ
を'true'
と比較しているので、中身が文字列であろうとbooleanであろうと文字列にキャストされて比較されると思います。 docs.github.com/ja/actions/learn-github-actions/expressions#contains== 'true'
を使っている限りではどっちでも安全なはず。(これが一番信頼できるはず)
すみません、これ完全に間違えてました。。。。。。。。。。型変換はされませんでした・・・・。
そしてgithub.event.inputs
で文字列だったbooleanは、inputs
でちゃんとしたbooleanになるっぽいです・・・。
https://github.com/Hiroshiba/test_actions/actions/runs/6486562288/job/17614989125
なので今のコードはバグっている状態になりそうです。 修正プルリクエストを投げます・・・。
内容
ビルド周りのgithubワークフローで、若干統一性がなかったのといくつか気になった点があったのでファクタリングしてみました。
github.event.inputs
はinputs
にできるので短くしたASSET_NAME
をステップ動的に定義するのではなく静的に定義できるようにした"contains(matrix.target, 'android')"
がエラーになっていたので直したSKIP_UPLOADING_RELEASE_ASSET
はもう存在しないのでコード中から消したenv.VERSION != '0.0.0'
はリリース判定に使わなくなったので変更した関連 Issue
その他
の前作業です。