-
Notifications
You must be signed in to change notification settings - Fork 196
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
リリースであればバージョンを追加し、それ以外であればlatestを追加する #290
Conversation
Pull Request Test Coverage Report for Build 1704606984
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
問題なさそうでしたが、もっと短く書けそうだったので別の案を出してみました。
今はreleaseとmaster push以外は動かないようにしているから大丈夫だけど、将来それ以外でも動くようになると予想外の動きをしそうです。(今もdispatchで変な挙動しそう)
どうにかして防げないでしょうか。
workflow_dispatch に関しては https://docs.github.com/ja/actions/learn-github-actions/events-that-trigger-workflows#viewing-all-properties-of-an-event を見てみたところ、手動実行時に直接入力を渡せるオプションがありそうでした。使えそうでしょうか……?
あ!将来このコードを変更する人が、latestとrelease tag以外で実行することを想定されていないコードだということに気づけないかもと思った次第です。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTMです!
内容
リリースであればバージョンを追加し、それ以外であればlatestを追加するようにします。
関連 Issue
fix #289
その他
今はreleaseとmaster push以外は動かないようにしているから大丈夫だけど、将来それ以外でも動くようになると予想外の動きをしそうです。(今もdispatchで変な挙動しそう)
どうにかして防げないでしょうか。