search

sounisi5011 / vec-draw

[WIP] SVGを書きやすくするDSL
0 stars 0 forks source link

scripts/commit-template.jsがcherry-pickに悪影響を及ぼす #8

Closed sounisi5011 closed 5 years ago

sounisi5011 commented 5 years ago

cherry-pickしようとしたところ、commitコメントにテンプレートの内容がそのまま入ってしまう問題が発生した。 cherry-pickする機会は少ないが、こんなややこしいバグは潰さねばならない。

sounisi5011 commented 5 years ago

COMMIT_SOURCEでちゃんと判定したほうが良さそうだ。

cherry-pickする場合、COMMIT_SOURCEの値は'message'になる。 これは、-mオプションなどを設定した時の値になる。 このような場合には、テンプレートの挿入は止めたほうがいいだろう。

そして、通常のcommitの場合は、COMMIT_SOURCEはundefinedになる。 そうか、commit-templateのあの条件分岐はそういう事だったのか…

sounisi5011 commented 5 years ago

-mオプションや-Cオプションのようなエディタが出てこないモードでテンプレートは挿入してはならない。コメントが解釈されない。

ただ、これらすべてをCOMMIT_SOURCEの値で判定できない。 エディタが出る-cオプションも、エディタが出ない-Cオプションも、COMMIT_SOURCEの値は"commit"。他の引数や標準入力にも差はない。

判定できるとすれば、コミットメッセージにデフォルトで入力されるコメントの有無。要検証