一年目の新人こそ、チーム１厳しいレビュワーになろう! - Qiita
はじめに LITALICO Engineers Advent Calendar 2023 の6日目の記事です。 昨日は、橋本さやかさんさんの、エンジニア組織のHRBPが行った「社内ラジオ」リニューアルPJTの具体的なプロセスと学びでした！ こんばんわ! LITALICO 新卒一年目の折田です！学校教育事業部でエンジニアをやらせていただいております。 現在、投稿日の前日19時。テスト前夜の如き焦燥感のなかでこの記事を執筆しております(社内のアドベントカレンダーのルールだと、投稿日の前日が締め切り) アドカレ初参戦なのですが、今回は、自分がPRレビューをするにあたって振り返ったことを書きたいと思います。 表題にもある通り、チーム１厳しくレビューしていた（はず）ので、今回は、新人が厳しいレビューをするにあたっての心構えや意味について書いてみました！（普段は大口叩くタイプじゃないので、いま震えなが
倍速視聴と良いレビュアーの条件と2022年のベスト反社会学講座ブログ
こんにちは、パオロ・マッツァリーノです。ここ2年ほどでしょうか、映画などを倍速で観る人が増えたそうで、その是非が議論されるようになりました。 私には、倍速で観る意味がよくわかりません。話題の作品を観てないといったらバカにされるから、ムリしてでも観たという実績だけは作っておきたい、みたいな心理なのかな？ でもおもしろい作品なら当然普通の速度で観ますよね。つまり、あまりおもしろくない作品を時短、タイパを優先して観るための手段なのでしょうけど、だとしたら理にかなってないんです。 たとえば2時間の映画は倍速で観ても1時間かかります。あまりおもしろくない作品を観る1時間は、やっぱりムダな時間でしかありません。 私の場合、映画やドラマは冒頭の15～20分を観て、おもしろくなりそうな気配がないな、自分好みのジャンルではないな、などと感じたら、見切りをつけてそこでやめてしまい、他の作品を観ます。よのなかに
なぜあなたのPull Requestは読まれないのか - Qiita
Pull Requestを出してレビューしてもらってから反映。 どこにでもあるありふれた開発フローに付きまとう、どこにでもあるありふれた問題。 「Pull Requestがレビューされない」 もちろん開発フローにレビューが含まれている以上、レビューをしないメンバーにも非がないとは言えませんが、多くの場合はレビューされないPRには問題があるものです。 デカい 兎にも角にもデカいPRは読むのがつらいです。 もちろん要件が明記されていないなど、他にもPRが読みにくくなる原因はたくさんありますが、一番はこれです。 極端な話、1行変更のPRは他に何も書かれていなくても実装内容を察することができますが、10ファイル100行の差分と箇条書き20点の要件が書かれたPRは内容を把握するだけで一苦労です。 しかし、このこと自体は数カ月でもコードを書いていれば自然と勘づくもの。 問題はなぜPRが大きくなってしま
クソレビューアだらけのレビュー会
体裁厨 「お♪ ここだけフォント違うくない? それからなんか間隔せまいし。」 用語統一厨 「"お問い合わせ"は"お問合せ"と表記することに決まってるので」 箇条書き過剰 「箇条書きにして。その方が分かりやすいよ。」 物忘れ激しい系 「こここんな仕様だっけ? え、設計書に書いてる? 作成者だれ? オレ? 決めた覚え無いけどなぁ…」 レアケース厨 「UUID? 100%衝突しないと言えない? じゃあダメじゃん。」 ショートカット厨 「Ctrl + Shift + T、Ctrl + Shift + T。あぁそれやるならCtrl + Shift + Rだ。」 遅れてくる系 「なんかここおかしくない? えっもう指摘された? ごめん、もう一回ちょっと説明して」 指摘曖昧系 「なんか分かりにくいなぁ。色付けたりするとかあんじゃん?」 寂しがり 「ちょっとなんか寂しいな。ここ説明書き足したら。いやこれだけ
仕様や実装方針の相談をPullRequestで行う取り組み - $shibayu36->blog;
これまで少し大きめな機能であれば、コードを書く前にまず仕様や実装の方針をissueのdescriptionにまとめ、それを先にレビューしてもらってから実装にとりかかるということをしていた。最近、その方針をそもそもrepositoryのファイルとして書いて、PullRequestしてレビューしてもらうようにしたら便利だったのでご紹介。 なぜコードを書く前に仕様や実装の方針をレビューするのか 前提としてなぜコードを書く前に仕様や実装の方針をレビューして欲しいのかについて書いておく。これはとにかく手戻りの量を少なくしたいという要求のためである。 実装に取り掛かろうとすると、実装の方針はいくつか思いつくが、どれが一番よいか迷うことはよくある。その場合に誰にも相談せず自分だけで判断し、コードを書いてからPullRequestを送った場合、もしその選択に重大なミスがあった場合全て書きなおさないといけな
コードレビューのベストプラクティス | POSTD
Wiredrive では、私たちはかなりの数のコードレビューを行います。しかし、ここで働き始める前には私はコードレビューなどしたことがありませんでした。今回は、私がコードレビューをする時に何に注目するようにしているかや、私の考え出したベストなコードレビューのやり方をお話したいと思います。 コードレビューとは、簡単に言うと2人以上の開発者で問題を引き起こしそうなコードの修正について話し合うことです。コードレビューをすることのメリットについては多くの記事で語られており、知識を共有できること、コードのクオリティが上がること、開発者が成長できることなどが挙げられています。しかし、レビューを行う上で、どのように進めていくかという具体的なことについてはあまり多く語られてないように私は思いました。 レビューで何に注目するか アーキテクチャ/デザイン 単一責任原則 : 1つのクラスは変更する理由が2つ以上
アプリ開発の品質底上げ施策をWebhooksでBotが支援する世界 - クックパッド開発者ブログ
こんにちは。技術部の松尾(@Kazu_cocoa)です。 主にモバイルアプリ開発において、数ヶ月前よりGitHubのWebhooksを使ったとある取り組みを始めました。HipChatやSlackなどをはじめとした様々なサービスとの連携サービスを提供しているGitHubですが、Webhooksの機能を使うことで、より自分たちの開発を支援する未来を創造できればと思います。以降では、実際の使用例、その実装よりな話しへと話しをすすめます。 クックパッドにおけるWebhooksの使用例 チェックリストによるセルフチェックをPR時に実施する モバイルアプリ開発はWebアプリ開発と異なる点が多々あります。例えば、開発対象の面では端末の多様性、端末システム側設定・通信状態の多様さなど、リリースの面ではデプロイに対する制限や更新がユーザ依存であることなど、です。そのため、当たり前品質の底上げのために不具合の
レビューポエム “関数型プログラミングに目覚めた! IQ145の女子高校生の先輩から受けた特訓5日間” を読んで
レビューポエム “関数型プログラミングに目覚めた! IQ145の女子高校生の先輩から受けた特訓5日間” を読んで 大阪天満橋のジュンク堂書店で購入しました。本当は立ち読みですませる気だったんですが、買わないでレビューはフェアではないかなと思い……今は後悔しています。 僕はプログラミングをしてご飯を食べていますが、関数型プログラミングの領分はまったく未経験です。 関数型の説明が適切かどうかの判断についてはできません。その上で、なぜ評価を低くしたのか、技術書的な側面とラノベ的な側面について分けて記述していきます。 技術書として まず、最初の「人物紹介」に記載されている、本書を通じての指南役である「サクラ」の説明に”プログラミング・スキルは、『神の目』と呼ばれる全能レベルにまで到達していると噂されるが、サクラのコードを読み解ける部員がいないため、真相は不明である”と記載があります。 他の人が読み
NECのWiMAXルーター「Aterm WM3500R」、駆動時間や使い勝手を徹底チェック - 日経トレンディネット
前回掲載した「NECのWiMAXルーター『Aterm WM3500R』、通信速度や実用度を徹底検証」に続き、最大通信速度40MbpsのモバイルWiMAXに対応したモバイルWi-Fiルーター「Aterm WM3500R（以下、WM3500R）」を実機でレビューしていきたい。前回は通信速度を中心にチェックしたが、今回はバッテリーの持ち時間や設定の柔軟性などの面をチェックしていきたい（編集部注：今回の検証に利用したのは、2010年12月末に公開された新ファームウエアを適用していない旧ファームウエアの製品です。ご了承ください）。 8時間のバッテリー駆動で、1日持ち歩いても余裕あり 最長8時間のバッテリーもWM3500Rの魅力のひとつだ。ノートパソコンをWM3500RにWi-Fi接続し、400kbps程度のストリーミング映像を再生し続けたところ、ほぼスペック値通り約7時間50分動作した。長時間連続通
1
リリース、障害情報などのサービスのお知らせ
最新の人気エントリーの配信
j次のブックマーク
k前のブックマーク
lあとで読む
eコメント一覧を開く
oページを開く
ReviewMe
Why I changed the way I think about Code Quality
GitHub - cookpad/dokumi: Automatically check if anything is wrong with the code in a pull request
窓の杜 - 【特集】オンラインストレージサービスを使いこなそう！
www.zumodrive.com - Similar Sites and Reviews | Xmarks