[B! review][source] masterqのブックマーク

Googleがコードレビューにおいてスムーズにコミュニケーションを進めるための注意点を公開

ソフトウェアの品質を保つためのツールとして、コードの変更点を一度他の人にチェックしてもらうという「コードレビュー」は高く評価されていますが、そのコードレビューにおいて、スムーズにレビューを進めるためのヒントをGoogleがブログに公開しています。 Google Testing Blog: Code Health: Respectful Reviews == Useful Reviews https://testing.google blog.com/2019/11/code-health-respectful-reviews-useful.html ◆レビューする場合とされる場合の両方において守るべきこと・相手の能力を尊重する人が違えば得意分野や背景知識なども異なります。相手には高い能力があるはずだという前提に立ち、まずは質問を通して理解を深めることが大切です。・根拠を提示するベスト

masterq 2019/11/15

リンク

人間らしくコードレビューするには(パート2) | Yakst

コードレビューの際に落とし穴にはまらずに、どのようにうまくコミュニケーションをとるか、に関する記事の後半部分です。著者がコードレビューで失敗した実例を元にお互いの衝突を避けるテクニックについて紹介します。 [プログラミング]原文 How to Do Code Reviews Like a Human (Part Two) - Silly Bits (English) 原文著者 Michael Lynch 原文公開日 2017-11-09 翻訳依頼者翻訳者 taka-h 翻訳レビュアー doubl emarket 原著者への翻訳報告 2151日前メールで報告済み 2149日前原著者承諾済み編集この記事はコードレビューの際に落とし穴にはまらずに、どのようにうまくコミュニケーションをとるか、に関する記事の後半部分です。今回はひどい衝突を避け成功裏にレビューを終わらせるテクニックに焦点をあ

masterq 2017/12/25

紳士的であることは僕にいつも難しいようです。。。

リンク

Re:VIEWでシンタックスハイライト - Qiita

HTMLの場合はそのままでいいが、PDFを生成するとjlisting.styがないというエラーが出るので、jlisting.styを落としてきてstyディレクトリに放り込む。シンタックスハイライト言語を追加する方法 rougeやpygmentsについては不明。ただし、開発が活発に続いておりたいていの言語には対応しているので困る場面は少ないだろう。 listingsはサポートしていない言語が多い。特に、最近の言語となると一切サポートがないので、シンタックスハイライトする言語を追加する必要がある。 texstyleに指定されたstyファイル(デフォルトだとreviewmacro.sty)に次のように書けばよい。他のファイルに分割したければusepackageを使う。「いろいろ書く」の部分はlistingsで言語を追加する方法を調べるとわかる。 C#問題 Re:VIEWでHighlight

masterq 2017/10/26

使わねば

リンク

人間らしくコードレビューするには(パート1) | Yakst

自動化することによりあなたはレビュアーとしてより価値のある貢献ができるようになります。importsの順序やソースコードのファイル名の命名規約などの問題を無視できるならば、機能上の誤りや可読性の問題といった、より関心のある問題にフォーカスすることができます。オーサーもまた自動化の恩恵を受けます。ケアレスミスを見つけるのに1時間浪費することなく、即座に見つけられます。即座にフィードバックを受けられることで、関係のあることがオーサーの頭に残り、これにより学習が容易となり、修正コストが低くなります。それに加え、彼らが初歩的な誤りについて指摘を受ける必要がある場合、あなたから指摘を受けるよりコンピューターから指摘を受けたほうが彼らの自尊心の観点からはるかに気分がよいわけです。これらの自動チェックはコードレビューのワークフローの中に入れましょう(例えば、Gitのコミット前のフックやGithubの

masterq 2017/10/25

熟読しよう。。。

リンク