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