-
Notifications
You must be signed in to change notification settings - Fork 10
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
バージョン情報の記載のあるheaderを作る #170
Comments
場所的には |
バージョンしか書かれてないヘッダのほうが,toolなどで読むときとか,自動で値更新するとき(があるかわからんが)に便利そうって思ったりはした.(あと,適当な場所で使うときにincludeするときとかもわかりやすい.バージョン番号つかうのだけに |
はそうなので,一旦それで,で,今後困ったらファイル分割するか~ |
toolで読む時,自動更新は別にどこにあってもそんなに変わらないかな,と思っています. |
実際,これはほぼなさそう? commit hashはtlmにいれたり,デバッグ表示にいれたりするので使うけど |
そうなんですよね.そんなにプログラム中で使う気はしないというのはある. |
じゃあ一旦 あとはどう更新するかか.main update prでもいいが,そうするとmain mergeではじめて上がった記載ファイルをどうdevまで持ってくるかが課題. これ,devをmain merge後に毎回rebaseするのってどうなんだ?(まあいいのかな) |
dev rebase って,PRでできるのか,というのも気になり |
これ何を気にしてます? |
手元でのforce pushにならない?って.(devとmainはいまforce push禁止してる |
あー,ff mergeのpr出せばいいのか? |
といってもいま,repository設定で--no-ffマージしか許されていない |
そうだった... > force push/--no-ff |
じゃあmain update PRでやればいいのでは |
あーーー,頭がついてなかった. そっか,main update prってdevにcommitできるじゃん. |
実はそう |
あれ,devはpush禁止してるので,commitできない説(PRマージしかできない) |
アッ |
の2段??...というリリースフロー? |
dev push解禁しない限りそれしかないかあ |
厳密にはtagよりも古いcommitにバージョン含まれちゃうが,まあ動作に影響のあるファイルには更新ないので,いいか. |
dev push解禁はないなぁ |
ではそれで行くかぁ. #169 とあわせて,リリースフロー,ドキュメント化します. |
ですよねー.adminだけpush可みたいなかんじにできればまた別なんですが. |
まあ,それもそれで事故こわいんで,一旦これで運用しよう,でクソだるかったらadmin解禁含めて考えよう. |
人間ではなくリリース担当Botを生やしてそいつにやらせる,などはないではない(が,運用コストを考えるとノーチャンな気がする) |
https://github.com/ut-issl/c2a-core/blob/develop/Docs/General/release.md でやった. 更新については,やってみてだるかったらまた考える. close. |
概要
バージョン情報の記載のあるheaderを作る
詳細
みたいな
close条件
できたら
備考
議論点は
関連
The text was updated successfully, but these errors were encountered: