回答編集履歴
4
言い回しを修正
test
CHANGED
|
@@ -5,7 +5,7 @@
|
|
|
5
5
|
また、相手の感情を不要に逆撫でする可能性がある文書は仕事である以上、関係値に依らず控えた方が良いと感じました。
|
|
6
6
|
『このような根拠でこう思いましたが、いかがでしょうか？』
|
|
7
7
|
あるいは断定できる場合
|
|
8
|
-
『このような根拠で修正が必要
|
|
8
|
+
『このような根拠で修正が必要そうです』
|
|
9
9
|
|
|
10
10
|
上記のような言い回しをする事で、相手も根拠が分かれば修正し易いですし成長にも繋がり信頼を損なう可能性は少ないと思います。
|
|
11
11
|
私がいままで本当に優秀だと感じた開発者は、コミュニケーションにおいても上記のような配慮が自然と出来る方が多かったです。
|
3
言い回しを修正
test
CHANGED
|
@@ -17,7 +17,7 @@
|
|
|
17
17
|
|
|
18
18
|
> PRを上げる前に改善が必要だと思ったところは、再度確認をしてからPRするようにお願いいたします。
|
|
19
19
|
|
|
20
|
-
→こちらも同時に修正根拠となるコーディング規約や参考コードが提示されている、あるいはタイポや不具合が見つかる場合の指摘としては正しいと思います。
|
|
20
|
+
→こちらも同時に修正根拠となるコーディング規約や参考コードが提示されている、あるいはタイポや不具合が繰り返し見つかる場合の指摘としては正しいと思います。
|
|
21
21
|
トリッキーなコードや命名センスなどについては努力でカバー出来るため、リーダブルコードなど学習リソースを提示できると親切だと思います。
|
|
22
22
|
(こちらもレビュアーが負担できるキャパシティを外れる場合は、採用が失敗している可能性があります)
|
|
23
23
|
|
2
不要な文書を削除
test
CHANGED
|
@@ -1,5 +1,5 @@
|
|
|
1
|
-
>
|
|
1
|
+
> このようなコードの書き方は問題ありですね。
|
|
2
|
-
>
|
|
2
|
+
> 複雑で可読性も非常に悪いコードです。修正をお願いいたします。
|
|
3
3
|
|
|
4
4
|
→実際はもう少し踏込んだコメントなのかもしれませんがコードの文脈を踏まえたとしても、具体的な悪い根拠が汲み取れないため修正が難しいと感じてしまいました。
|
|
5
5
|
また、相手の感情を不要に逆撫でする可能性がある文書は仕事である以上、関係値に依らず控えた方が良いと感じました。
|
1
言い回しを修正
test
CHANGED
|
@@ -1,5 +1,5 @@
|
|
|
1
1
|
> 引用テキストこのようなコードの書き方は問題ありですね。
|
|
2
|
-
複雑で可読性も非常に悪いコードです。修正をお願いいたします。
|
|
2
|
+
> 引用テキスト複雑で可読性も非常に悪いコードです。修正をお願いいたします。
|
|
3
3
|
|
|
4
4
|
→実際はもう少し踏込んだコメントなのかもしれませんがコードの文脈を踏まえたとしても、具体的な悪い根拠が汲み取れないため修正が難しいと感じてしまいました。
|
|
5
5
|
また、相手の感情を不要に逆撫でする可能性がある文書は仕事である以上、関係値に依らず控えた方が良いと感じました。
|
|
@@ -13,7 +13,7 @@
|
|
|
13
13
|
逆に相手との関係性を壊すような物言いしかできない方は経験上直ぐにチームから離れる、もしくはチームから人が離れてプロジェクトが失敗します。
|
|
14
14
|
|
|
15
15
|
チームギークという書籍にHRT(謙虚、尊敬、信頼)という言葉が出てきますが、この事例にも適用されると思います。
|
|
16
|
-
(ただ、レビュイーに関してあまりにも仕事にならず努力も見られない場合は論外
|
|
16
|
+
(ただ、レビュイーに関してあまりにも仕事にならず努力も見られない場合は論外だと思います)
|
|
17
17
|
|
|
18
18
|
> PRを上げる前に改善が必要だと思ったところは、再度確認をしてからPRするようにお願いいたします。
|
|
19
19
|
|