タグ

PRとコードレビューに関するbraitomのブックマーク (2)

  • レビューしやすいプルリクエスト | DevelopersIO

    普段レビューをしていて、レビューしやすいプルリクエストに対して個人的に感じている特徴をまとめてみました。 普段レビューをしていて、レビューしやすいプルリクエストに対して個人的に感じている特徴をまとめてみました。 割と大きめなソースコードに対するレビューの話が主となります。 ざっくりまとめ 記事では以下のようなトピックについて記載しています。 差分の目的が1つ レビューをしながら「私はいま何のレビューをしているのか」のコンテキストスイッチが発生しないので嬉しい 何を達成したいのかがわかる レビューの多くは「やりたいこと」と「実現方法」のすり合わせなので、前者の精度を上げたい 分割されすぎていない 他のコードとの関連性や構造についてのレビューがしやすい レビューの強弱をつけるための情報がついている 機械的な変換の差分だったりした場合、それが事前にわかると嬉しい 検証結果が書いてある コードだ

    レビューしやすいプルリクエスト | DevelopersIO
    braitom
    braitom 2020/07/16
    レビューしやすいPRについて。差分の目的が1つ、分割されすぎていない、ファイル間の関連が分かるなど。
  • よりよい機能をより早く届けるためにGit/GitHubを使いこなす10の方法 - Qiita

    この記事は freee Engineers Advent Calendarの6日目です。 こんにちは。freee株式会社でアプリエンジニアをしている @kompiroです。主に会計freeeの開発に携わっています。 僕には「よりよい機能を、より早くユーザーに届けたい」という信念があります。今日はよりよい機能を、より早くユーザーに届けるためのGit/GitHubを活用する10の方法と題してGitGitHubの便利な使い方や日々のTipsをお届けします。1 PRのレビュー完了までの時間を最速にするための工夫 freeeではGitHub上でレビューをしながら開発を進めています。レビューをするのもされるのも、うまく進めないと結構時間がかかります。もちろんコードをより良くする作業に時間を割いているのであれば時間がかかるのも致し方ないです。しかし、進め方による問題でも数日〜数週間デプロイが延期されれ

    よりよい機能をより早く届けるためにGit/GitHubを使いこなす10の方法 - Qiita
    braitom
    braitom 2016/12/10
    PRに対するレビューを効率的におこなうためのTipsが書かれている。レビューする側、レビューを出す側両方の視点が書かれていて非常に参考になる。便利ツールの紹介も書かれている。
  • 1