アカウント名:
パスワード:
プルリクエストは受け付ける側の負担を考慮して出すべきだと思います
たとえばコミットメッセージには(1)変更の概要(何を変更したか)(2)変更の理由(何故変更したのか)の両方を書くべきです
受け付ける側の立場からするとこの2つの情報が揃って初めて merge してみるかという気になります
タレコミのプルリクエストの例では(2)が欠けており変更の必要性が伝わってきませんこれでは無視されたり却下されても文句は言えないと思います
>「『ほどほどに』という単語は『ほど』がduplicateして間違っているので修正しました」それをなぜ間違いとしているかが書かれてないから(2)の要件を満たしてないと言ってるように見えるけどちゃんと読めてます?
Linux Torvalds氏がスラドを読まない理由
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
あつくて寝られない時はhackしろ! 386BSD(98)はそうやってつくられましたよ? -- あるハッカー
悪いリクエストの典型例 (スコア:5, 興味深い)
プルリクエストは受け付ける側の負担を考慮して出すべきだと思います
たとえばコミットメッセージには
(1)変更の概要(何を変更したか)
(2)変更の理由(何故変更したのか)
の両方を書くべきです
受け付ける側の立場からすると
この2つの情報が揃って初めて merge してみるかという気になります
タレコミのプルリクエストの例では(2)が欠けており
変更の必要性が伝わってきません
これでは無視されたり却下されても文句は言えないと思います
Re: (スコア:0)
> 変更の必要性が伝わってきません
そういう問題だろうか。
あなたは「『ほどほどに』という単語は『ほど』がduplicateして間違っているので修正しました」、というpull requestを「mergeしてみるか」という気になるのでしょうか。
Re: (スコア:0)
>「『ほどほどに』という単語は『ほど』がduplicateして間違っているので修正しました」
それをなぜ間違いとしているかが書かれてないから(2)の要件を満たしてないと言ってるように見えるけど
ちゃんと読めてます?
Re:悪いリクエストの典型例 (スコア:0)
Linux Torvalds氏がスラドを読まない理由