タグ

PRレビューに関するmokuozのブックマーク (2)

  • プルリクエストへのコメントでは Text Blaze でラベルバッジをつけよう! - Qiita

    [imo] enumのswitch文ではdefaultは使用せずに全て網羅して記述した方が良いと思いますがいかがでしょうか。 将来的にcaseが追加され、そのcaseが網羅されていない際にコンパイラが検知してくれるためです。 defaultを使用した場合は、該当ケースが網羅されていなくてもそのケースはdefaultにながれてしまい、 バグが生まれる原因になりかねません。 ②ラベルを視認しやすくするためにバッジにする ①の方法をおこなうことで改善できるかと思いますが、より視認性をあげることができそうです。 たとえば、バッジ画像をつかう方法があります。 例えば、①のmustの場合は下記のようなものを使用します。 行なっていることは、Markdownの画像付与です。 ![review:must](https://img.shields.io/badge/review-must-red.svg)

    プルリクエストへのコメントでは Text Blaze でラベルバッジをつけよう! - Qiita
  • もう初回コードレビューはAIに任せる時代になった - CodeRabbit -

    どんな人向けの記事? レビューによって心理的なダメージを受けやすい方 非エンジニアだが、エンジニアチームがどんな機能を作っているか知りたい方 業務が溜まっていて、レビューに割く時間を捻出するのに苦労している方 コピペできるコードも公開します 初回レビューをAIに任せると、いろんなロールの人の役に立つ レビューは得意ですか? 優秀なエンジニアしかいないチームであれば、PRは1トピックに絞って小さく明確なコミットによって作成され、適切な要約とともに提供されることでしょう。 しかし、実際にはいろいろな制約から、PRが想定よりずっと大きくなってしまったり、関連トピックと異なるコードが混じってしまうこともあります。 実際のところ、大きなPRを適切にレビューするのは難しいことです。また、自分が詳しくない領域のレビューを行わなければいけない機会もあります。 今回の記事は、レビューを作成してくれるAI C

    もう初回コードレビューはAIに任せる時代になった - CodeRabbit -
  • 1