[B! レビュー] tajima0111185のブックマーク

［速報］「Amazon CodeGuru」発表。機械学習したコンピュータが自動でコードレビュー、問題あるコードや実行の遅い部分などを指摘。AWS re:Invent 2019

Amazon Web Services（AWS）は、米ラスベガスで開催中の年次イベント「AWS re:Invent 2019」の基調講演で、機械学習を用いて自動的にコンピュータがコードレビューをしてくれる「Amazon CodeGuru」を発表しました。 Amazon CodeGuruのコードレビュー機能は、Amazon自身のこれまでの大量のコードと、GitHubで公開されているポピュラーな1万のオープンソースソフトウェアのコードを基に機械学習のトレーニングを行ったモデルを用いて、対象となるコードを解析。 GitHubやCodeCommitのプルリクエストと連係し、問題があるとされた個所には人間に読める形式でコメントをしてくれるというもの。並列処理や脆弱性の問題あるコードを指摘例えばAWSにおけるベストプラクティスのコードから外れているものや、並列処理における問題などの指摘。

tajima0111185 2019/12/05

リンク

入社からの半年間でコードレビューで指摘されたことのまとめ - 30歳からのプログラミング

実務未経験でプログラマとして入社して半年以上が経った。コードレビューで指摘されたことを備忘録としてまとめておく。自分なりにまとめたものなので、レビュアーが言いたかったこととニュアンスや解釈がずれている可能性はある。初歩的な内容ばかりで我ながらうんざりする。せっかく優秀な同僚ばかりなのだからもっと高度なことを学びたいが、こういう初歩的なことが出来ないのが俺の現状なのだから、仕方ない。そもそもPullRequestを送ったこともなかったわけだし。入社初日は、一人でPullRequestの出し方を練習していた。それを考えればまあ、こんなものだろうか。当たり前のことをちゃんと当たり前に出来るようになって、早く、次のステージに進みたい。 PullRequest(PR) PRのタイトルは分かりやすいものに。必要に応じてチケットの番号なども入れる。コミットやPRは出来るだけ粒度を細かくす

tajima0111185 2017/08/28

リンク

コードレビューを会話しながら行う取り組み - Hatena Developer Blog

こんにちは。アプリケーションエンジニアの id:itchynyです。今回は、コードレビューを会話しながら行う取り組みについて紹介します。コードレビューは大事なコミュニケーションの場です。コードレビューの効用としては、単純なミスがあるコードをリリースしない・プロダクトのコードの品質をよりよくしていく、あるいはその方策を模索するといったことが挙げられます。こういったことは当然のことですが、なによりもまず、レビューというのは一緒にプロダクトを作っている仲間とのコミュニケーションの場だと思います。多くの人は、プロダクトのコードをよくしていきたい、読みやすいコードを書きたい、分かりやすいコードで目的の機能を作りたいといった共通の思いを持っていることでしょう。コードを書いた人の思いを汲み取りながら、共感したり、譲歩したりしながら、よりよい方法を提示していきます。それでも時には、どういうコ

tajima0111185 2017/06/23

レビュー

リンク

Pull Requestに潜むタイポを自動的に検出し、修正を代行するBot - Qiita

いざPull Requestのレビュー！と挑んだ瞬間、「ここタイポな」という先制パンチをくらうのはとても残念なことです。また、これは指摘しているほうにとってもチェックが負担で、気が重いものです。人間は人間にしかできないチェックに集中すべきですし、貴重なレビュー時間を誤字脱字の修正に使うのはもったいないです。そこで開発したのが、タイポの自動検知と修正を代行するBot。その名もtypotです。 chakki-works/typot こちらは先日公開がアナウンスされたGitHub Marketplaceと共に公開された、新しいGitHubアプリの形態であるGitHub Appsで作成しています(それまではWebhookかOAuthだった)。 GitHub AppsはOAuthのようにユーザーではなく、リポジトリにひもつく形態になります。そのため、管理者ユーザーがいなくなった(あるいは権限を失

tajima0111185 2017/05/31

リンク

コードレビューで気をつける言葉や行い - macotox’s blog

コードレビューにstashを使ってます。こいつはgitのブランチ間の差分に対してコメントをつけることができるツールです。ただ、ネットを介したコミュニケーションって何故か気が大きくなってしまったり、感情が見えづらかったりで誤解を生みがちです。特にコードレビューって間違いを指摘するとかあんまり楽しい会話をするわけでもないので、言葉には気を使わないといけません。今日は自分が気をつけている言葉や行いを上げてみます。否定しない def get_name(name) @user.find(name: name) end ☓：getは軽量なアクセッサとして使うのが常識なのでやめて下さい。 ◯：findしてることが分かるメソッド名が良いですいちいち否定する必要はないです。素直にどうして欲しいか書きましょう。否定しない2 「けど〜」 def search(name) @user.find(na