web-dev-qa-db-ja.com

他人の作業を元に戻すためのエチケット

最近、チームメイトと「元に戻す前に相談していない」との議論がありました。「馬鹿のように見える」からです。 (コンテキストについては、これは大学のプロジェクトであり、元に戻すのはコードの配置でした。)

これは私に不思議に思います:元に戻す必要があることをコミッターに知らせるための規範は何ですか?どのようにして、ハードフィーリングを扇動せずにこのコミットを元に戻す必要があると彼らに伝えますか?

8
tyteen4a03

コードレビューがあります。

これでほとんどの問題が解決されるはずです。コミットを使用している場合でも、チームで他の1人がレビューするだけであっても、プルリクエストを行うことができます。

ミスがあった場合、それはチームの責任であり、1人の責任ではありません。

何らかの理由でそれをしたくない場合は、簡単な方法はありません。リポジトリにコードを取り込むプロセスはありません。そのため、人々の気持ちが傷ついても驚かないでください。

よくあることですが、他の人を苛立たせるようなことをするような状況に遭遇したときは、行動を起こす前にまず彼らと話し合うのがよいでしょう。この場合、おそらく同僚と話して、「[理由]のためにXを元に戻すことを考えています-どう思いますか?」

11
enderland

あなたは彼のコミットを元に戻す正当な理由があったか、そうでなかった。正当な理由がある場合、「私を馬鹿のように見せる」というのは適切な反論ではありません。とはいえ、議論を避ける理由を最初にその人に知らせるのは礼儀でしょう。

とはいえ、コードレビューを行うほうがかなり良いので、元に戻せないことが望ましい-コミットが拒否されることはあるかもしれませんが、そのような状況に陥るべきではありません元に戻しました。

3
gnasher729

この問題の根本的な問題は、「コンポーネントを技術レベルで誰が所有するか」です。

これに対する答えがない場合、または回答者が「私たちはすべてやる」または「誰もやらない」である場合、またはあなたがただのぎこちない困惑した見た目になり、誰も問題を見ない場合、あなたのエネルギーを無駄にせず、別の仕事を見つけてください。

回答があり、チェンジャーと話し合っても合意に至らなかった場合は、所有者とチェンジャーと面会し、ケースを作成して所有者に決定させます。

責任の共有は、すべての嘆かわしいコードベースと惨めな従業員の根本にあります。

2
Martin Maat

誰かが何かにかなりの時間を費やしている場合、特にそれが最近の場合は、相談することを依頼するのは公平だと思います。最初にコンセンサスを得ることをお勧めします。理想的には、その人に自分自身を元に戻すよう説得するでしょう。

解決できないという意見の相違に遭遇した場合は、第三者に相談してください。過半数の投票を勝ち取るか、3人目がリード開発者であることを確認します。

時々、物事を手放す必要があり、すべての戦いに勝つ必要はありません。聞いたことを確認してください。

あなたが書いたコードに執着しないようにするのは健康だと思います。私は自分のコードを頻繁に元に戻します。他の人も、適切なときに同じようにできるはずです。

1
Joppe

単に元に戻すのではなく、自分で元に戻してもらいます。

復帰をリクエストするときに与えることができる一般的な理由:

  • 「コミットXYZはこれとそのユースケース/機能/テストケースを壊します」
  • 「コミットXYZは、間違ったインデントなどのコーディング基準に準拠していません。」
  • 「コミットXYZが組織のプロセスに違反しています。たとえば、コンポーネントAへのコミットは、コンポーネントの所有者Bがコミット前に確認する必要があります。」

自分で行っていない何かを元に戻す理由のみ

  • 人が離れている(休日/病気)
  • コミットによってビルドが壊れたか、他のチームメンバーが作業を行えないようにする何かが行われました。
  • バギーコミットはすでに本稼働中であり、できるだけ早く修正する必要があります
1
arved