気に入った記事をブックマーク
- 気に入った記事を保存できます
保存した記事の一覧は、はてなブックマークで確認・編集ができます
- 記事を読んだ感想やメモを書き残せます
- 非公開でブックマークすることもできます
エントリーの編集
エントリーの編集は全ユーザーに共通の機能です。
必ずガイドラインを一読の上ご利用ください。
記事へのコメント32件
- 注目コメント
- 新着コメント
gyampy
仕事ができるというのは、他の人が「何がわからないと思う」のかを知って、気を配るということなのだと思う。レビューをそのまま実行するんじゃなくて、その意味を理解しようとする姿勢を持ってる人は優秀
a-kuma3
裏側に "読みづらいコードには、たいてい地雷が埋まってる" という経験則があるからでしょう。LINQ (Lambda) に if の分岐があったら、普通の人は突っ込む。その Lambda のテストコードはないわけだから
khtokage
周りがバカに見えてるとき一番バカなのは自分という、いわゆる「バカの壁」に囲まれてる状態ですね。 「バカの壁」は一度克服しても何か学ぶ度に発生するのが難しい。周りが馬鹿に見えたら注意するしかないですね。
ka-ka_xyz
"レビューをそのまま実装しようと考えると、「コードは読み物」のマインドセットがあっても、コードが歪んでしまうことがある。""ゼロから考えて「読み物」のコードを書いていこう。" あー、わかるわかるこれはわかる
diveintounlimit
イケア効果かな。あと指示だけコメントしちゃうのは、それを書く側の問題もある。とりあえず記載内容からは、この人のITエンジニアとしてのスキルは自分で思ってるよりは低そうだ。
jiro68
毎日多くの PR が来る人にとっては1つ1つのPRを詳細にレビュー出来ないだろうから、下まで読まずに目に付いた事をコメントするのは理解出来る。プロなら分かりやすいコードを書けという事。コードは自分のものではない
NOV1975
文化がよくわからん。コードがポンコツすぎてせめて補助線引いてくれよじゃない限りレビュアーの責務ってコードの正しさを確認することだと思うので下まで読めよって思うが。（その上でコメント書いとけ、ならわかる
注目コメント算出アルゴリズムの一部にLINEヤフー株式会社の「建設的コメント順位付けモデルAPI」を使用しています
いまの話題をアプリでチェック！
- バナー広告なし
- ミュート機能あり
- ダークモード搭載
関連記事
Pull Request のコメント数を減らすアホみたいなコツ｜牛尾 剛
私は長年 Pull Request のコメント数が多くて何回もレビューを往復することが多くて大変つらかったが最... 私は長年 Pull Request のコメント数が多くて何回もレビューを往復することが多くて大変つらかったが最近ものすごく単純なコツに最近きづいたのでそのことをシェアしようと思う。 Pull Requestレビューの悩みこれはならない人はならないので、共感してもらえる人は少ないかもしれないが自分の悩みは Pull Requestのコメント数でこれが本当に多い。何がつらいって、レビューのコメントが多いという事は、マージに時間が掛かるということだ。最初にコードを書いてテストして完成させるのは２時間もかかってないのに大抵レビューで何往復もして時間を取られるのが本当につらいし、進捗がでないもの嫌だし、時間かかるし、自分が最近解決したい問題の中でも筆頭の問題だった。 何が悪いのだろう？すごく嫌なので物凄く考えたがうまくいかなかった。例えば、英語のスペルミスも良くしたし、ログやコメントの英文にレビュー
ブックマークしたユーザー
すべてのユーザーの
詳細を表示します
詳細を表示します
2023/07/07 リンク