Pull Request のコメント数を減らすアホみたいなコツ｜牛尾 剛
私は長年 Pull Request のコメント数が多くて何回もレビューを往復することが多くて大変つらかったが最近ものすごく単純なコツに最近きづいたのでそのことをシェアしようと思う。 Pull Requestレビューの悩みこれはならない人はならないので、共感してもらえる人は少ないかもしれないが自分の悩みは Pull Requestのコメント数でこれが本当に多い。何がつらいって、レビューのコメントが多いという事は、マージに時間が掛かるということだ。最初にコードを書いてテストして完成させるのは２時間もかかってないのに大抵レビューで何往復もして時間を取られるのが本当につらいし、進捗がでないもの嫌だし、時間かかるし、自分が最近解決したい問題の中でも筆頭の問題だった。 何が悪いのだろう？すごく嫌なので物凄く考えたがうまくいかなかった。例えば、英語のスペルミスも良くしたし、ログやコメントの英文にレビュー
機械学習でコードレビューを自動化 ! Amazon CodeGuru 3 分間クッキング - builders.flash☆ - 変化を求めるデベロッパーを応援するウェブマガジン | AWS
アプリケーション開発に携わっている方であれば、誰もがコードレビューを実施する、あるいはコードレビューを受けるという立場として、コードレビューを経験したことがあるのではないでしょうか。コードレビューのプロセスは、プロダクトの欠陥を無くすという観点と、組織としてのエンジニアリング力を維持するという観点で不可欠です。 しかし、コードレビューを行う人材の確保が難しいケースも存在すると思います。レビューを行うにあたっての知識を備えていることがまず前提としてあり、更にプロダクト全体に対する理解も必要とされます。そして、いざコードレビューとなると、時間を確保する必要があります。開発が活発なほどコードレビューの時間も比例して増えてしまうので、レビュアーの負担増にも繋がります。コードレビューがネックになってリリースが遅延してしまうのでは本末転倒ですよね。よって、コードレビューの自動化はこれまでも非常に注目を
レビューコメントにラベルをつけるだけで開発効率があがって幸せになれそうな話 - Qiita
はじめに コードレビューを受けている時,これは修正が必須なレビューコメントなのか,出来たら修正した方がいいものなのか,判断に迷う時ってありませんか？ 付き合いの長いチームや, メンバー同士でレビューをしていると感覚的に分かることもあるけど, 新しいチーム, 新しいメンバーなど関係性が出来上がる前だと, 温度感が分からず判断に迷うことがありそうです. 結果, 全てを対応しようと頑張りすぎてしまい, 修正に恐ろしく時間がかかってリリースが遅れるなど本末転倒な不幸なケースも起こりえそうです。 例）先輩がいってるし全部直さなきゃ...! [must] 必ず対応してほしい！ [imo] 自分の意見や提案・好み。 自分ならこう書くけどどうかな？ [nits] 些細な指摘。 ほんの小さな指摘だけど出来れば直してほしい。インデントやタイポ [ask] 質問,確認。 コードの意味や背景が分からないから教えて
CircleCIとdangerで、どんなプロジェクトでも簡単に「Stop saying "you forgot to …"」 - Qiita
CircleCIとdangerで、どんなプロジェクトでも簡単に「Stop saying "you forgot to …"」GitHubCircleCIpullrequestDangerCircleCI2.0 CircleCI Advent Calendar 2018の9日目です🎄 8日目の記事は @f-naoto832 さんのCircleCIで試行錯誤した話でした。 はじめに この記事は CircleCI2.0とdanger（ruby）専用のdocker imageを使うことで どんなプロジェクトでもdangerを利用できるようにした例を紹介します。 主な対象読者 最終的にPRで、approveする人（Techリードとか?） ruby, js, swift以外のどんなプロジェクトでもdangerを実行したい人 CIの整備をしている人 きっかけ ソースコードレビュー時、「〇〇、まだ終わっ
Introducing draft pull requests
ProductIntroducing draft pull requestsYou can now use draft pull requests to clearly tag when you’re coding a work in progress. At GitHub, we’ve always felt that you should be able to open a pull request to start a conversation with your collaborators as soon as your brilliant idea or code is ready to take shape. Even if you end up closing the pull request for something else, or refactoring the co
【前編】全部見ます！プロを目指すRailsエンジニアのための公開コードレビュー #railsdm
Rails Developers Meetup #4 でお話しした「プロを目指すRailsエンジニアのための公開コードレビュー」の続編として、すべての解答にコメントを入れてみました。 長いので前編と後編に分けてあります。 【前編】 - この動画の注意事項等 - 発表当日の内容のおさらい - #9 ebihara99999 - #10 ttwo32 - #11 cohakim - #12 sanfrecce-osaka - #14 yamotonalds - #16 ota42y - #17 itume - #18 yusuke319 - #19 bellwood4486 【後編】 https://youtu.be/yDxg0sXHXOo - #20 tsumuchan - #21 toyokappa - #22 ryu39 - #23
Gerrit Code Review | Gerrit Code Review
Discuss code Read old and new versions of files with syntax highlighting and colored differences. Discuss specific sections with others to make the right changes. Manage and serve Git repositories Gerrit includes Git-enabled SSH and HTTPS servers compatible with all Git clients. Simplify management by hosting many Git repositories together. Schedule git gc over all managed repositories and replica
prprでGithubのPullRequestレビュー依頼をSlack通知する - hitomedia Tech Blog
最近サーバサイドをやりつつiOSアプリ開発をやったり何でも屋になっている hilotter です。 GithubのPullRequestレビュー機能、便利ですね。 チーム開発においては相互レビューが非常に大事です。 今回は、レビュー依頼を行った際にSlackに自動通知するようにしたらより便利になったというお話を共有させていただきます。 SlackのGitHub連携の課題 Slack公式のGithub連携がありますが、こちらは以下の2点が課題となっていました。 1. メンション付きのコメントをしても、message-attachmentsの記法になっているため、うまく通知されずコメントを見落としてしまう このため、コメント後にSlackでも「コメントしました！」と2重で連絡をする必要がありました。 2. PullRequestのレビュー依頼を行ってもレビュアーへの通知が飛ばないため、レビュー
dangerをCircleCI上で使ってPull Requestを精査する - Qiita
Dangerとは Dangerとは、ざっくり言うと、PRのレビューをするときに、"You Forgot To...(...するのを忘れてませんか？）"と指摘するのを自動化するツールらしいです。 似たようなツールとして、以前から packsaddle/ruby-saddler を使っていたのは、 Androidのコードを自動で解析し、GitHubのpull requestにコメントする - Qiita にも書いていました。 Android開発のコードレビューbotを乗り換えた話 - クックパッド開発者ブログ を読むと、クックパッドさんもDokumiから乗り換えたということで、使ってみました。（ミーハー） （ミーハーついでに、fastlaneとか作ってる Felix Krauseさんが絡んでる？というのも、やってみた要因の一つ） 上記のブログにもDokumi時代の問題点が書かれていましたが、私
Pull Requestに潜むタイポを自動的に検出し、修正を代行するBot - Qiita
いざPull Requestのレビュー！と挑んだ瞬間、「ここタイポな」という先制パンチをくらうのはとても残念なことです。 また、これは指摘しているほうにとってもチェックが負担で、気が重いものです。 人間は人間にしかできないチェックに集中すべきですし、貴重なレビュー時間を誤字脱字の修正に使うのはもったいないです。そこで開発したのが、タイポの自動検知と修正を代行するBot。その名もtypotです。 chakki-works/typot こちらは先日公開がアナウンスされたGitHub Marketplaceと共に公開された、新しいGitHubアプリの形態であるGitHub Appsで作成しています(それまではWebhookかOAuthだった)。 GitHub AppsはOAuthのようにユーザーではなく、リポジトリにひもつく形態になります。そのため、管理者ユーザーがいなくなった(あるいは権限を失
リリース、障害情報などのサービスのお知らせ
最新の人気エントリーの配信
j次のブックマーク
k前のブックマーク
lあとで読む
eコメント一覧を開く
oページを開く
GitHub Next
Dangerでpull requestレビューの指摘事項を減らす
LTTM
GitHub - kirillrogovoy/pullkee: A simple Pull Requests analyzer
All-in-one visual testing and review platform | Percy
GitHub - bors-ng/bors-ng: 👁 A merge bot for GitHub Pull Requests
How to change the base branch of a pull request?
PRする前にやる事リスト.md
GitHub - MysticDoll/alert-before-pullreq: プルリク前に許可を取る必要がある方全てのために
GitHub - speee/webapp-revieee
