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


8

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

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


その人に「このコードをこのように変更してもよろしいですか...」と尋ねます。
ブライアンオークリー

回答:


11

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

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

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

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

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


とはいえ、私の経験では、大学のプロジェクトのコードレビューを行うことは非常に難しい場合があります。大学での私のプロジェクトはすべて、全員を同じ部屋に入れる時間を見つけるのに苦労しました!業界では簡単ですが、そのようなレビューのメリットを確認するために説得力のあるマネージャーを必要とする場合があり、それでもすべてのコミットでレビューがあるとは限りません。
Kat

とは言っても、大学のプロジェクトでの私の経験は、一貫したスタイルでコードを記述したり、非常にバグの多い実装を記述したりすることさえできなかったため、他の人々の変更を何度も上書きしました。ほとんどの大学のプロジェクトでマネージャーがいないことは、人々がうまく連携できるようにするための大きな制限です(業界での経験がなく、上手に書かれたコードが理解されていない多数の学生は言うまでもありません)。
Kat

3

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

とは言っても、コードレビューを行う方がはるかに良いので、うまくいけば元に戻せないはずです-コミットが拒否されることはあるかもしれませんが、元に戻された状況に陥るべきではありません。


「私を馬鹿のように見せる」というのは良い反論ではありません。合理的な職場環境では、同僚に親切になりたいと思っています。これに対処する方法は、自分のものではないコードをさりげなく戻すよりも多くあります。
thedayturns

2

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

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

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

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


1

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

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

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

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


1

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

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

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

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

  • 人が離れている(休日/病気)
  • コミットによりビルドが壊れたか、他のチームメンバーが作業を行えないようにする何かを行います。
  • バギーコミットはすでに本稼働中であり、できるだけ早く修正する必要があります
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.