タグ

reviewに関するama-chのブックマーク (7)

  • 「速」を落とさないコードレビュー

    Forkwell Meetup #3 https://forkwell.connpass.com/event/48147/Read less

    「速」を落とさないコードレビュー
  • 気が狂った設計 - hitode909の日記

    大きめのこととか,自信のないところを触るときは,コード書く前に,こういう作戦考えてみたけどどうですかって聞いてみたり,こういうことやりたいんだけど一緒に考えませんかって,いっしょに話して設計考えたりするとよいと思う. 一緒に考えたすぐあとに気が狂った設計とか言い出したらおかしいので,未然に変な設計のままコード書いてしまうのを防げる. 特に辛い気持ちになるのが、「気が狂った設計」「クソコード」「(こんな実装は)有り得ない」といった言葉だ。 Pull Requestのレビューが辛くて会社をやめたい 単に言葉が強いのはよくないと思う.我が社にはそんな強い言葉でレビュー書く人はいない. 我が社には,普段から強い言葉を発する人もいなくて,みんな物腰柔らかな変な言葉を話している. 言葉使いや文体は,ずっと過ごしてると同僚から移ったりするので,普段からそういう言葉を話していると,全体の雰囲気も悪くなりそ

    気が狂った設計 - hitode909の日記
    ama-ch
    ama-ch 2015/07/04
    誰だって強い言い方をされたら傷つくし、わざとクソコードを書く人なんていないんだから、批判の前に相手を尊重する気持ちを持ちたい。
  • Reputation Management Software to increase brand trust | Review Ninja

    Grow your business with Customer engagementReview Ninja integrates with social media accounts and makes communication between businesses and consumers hassle-free and effective. Take control of customer interactions with tools that increase brand trust and exposure on popular review sites. A simple, user-friendly dashboard provides real-time updates on listings, interactions, and rankings. We help

  • GitHubと連係して動作するコードレビューツール「ReviewNinja」。あの欧SAPがオープンソースで開発中。しかもMEANスタック使用

    GitHubと連係するコードレビューツール「ReviewNinja」がGitHub上で公開されています。開発しているのは欧SAP。あの業務アプリケーションで知られるSAPです。 Review Ninjaはまだ開発の初期にあるようですが、GitHub上で次のように解説されています。 ReviewNinja is a lightweight code review tool that works with GitHub, providing a more structured way to use pull requests for code review. ReviewNinjaは軽量なコードレビューツールで、GitHubと連係しています。これはコードレビューの場面でプルリクエストを用いてより構造的な方法を提供するものです。 (一部省略) ReviewNinja defines a clea

    GitHubと連係して動作するコードレビューツール「ReviewNinja」。あの欧SAPがオープンソースで開発中。しかもMEANスタック使用
  • 眼鏡なしのコードレビュー | POSTD

    例えば、あなたが驚くほど聡明な開発チームのメンバーで、コードレビューのみに一日の時間を確保しているとします。しかし作業を開始して2時間後、眼鏡を忘れてきてしまい、午前中はぼんやりとしたカラフルな表示を見つめていただけだったということに気づいたとします。さて、あなたはどうしますか? 家まで歩いて10分もかからないし、天気も良ければ、眼鏡を取りに帰るのが一番です。でも朝家を出るとき、攻撃的なスズメバチの群れが眼鏡の置いてある部屋に巣を作って、邪魔されたくない様子だったらどうしますか? そういう時はもちろん、コンタクトレンズを付けてきたふりをして、恥ずかしい思いをしないようにするのがよいでしょう。実際に読むことなく膨大な量のファイルを見分けることができるということを覚えておいて下さい。 参考コード 1 不安の種は隔離するべきだということに誰も異論はないでしょう。そしてもちろん、あらゆるクラスは一

    眼鏡なしのコードレビュー | POSTD
  • 雑なレビュー - ✘╹◡╹✘

    背景 レビューに時間を掛けているわりにあまり成果が出ていない 問題意識を感じる レビューという行為にもっと周りから理解があればいいのに、という風に考える 原因を外部に求めるのは良くないなと考え直す これまで自分が発言したコメントを読み返す 過度にフォーマル過ぎたり、コードの表層しか指摘していないところが多々あることが分かる 問題 GitHubのPull RequestやIssueでのコメントに、出来るだけ間違いや誤解が無いように気を付けられた丁寧な文章で書いてしまうことが多い。しかしながら、もっと普段互いに会話するときに使うような雑な言葉で書いて、コミュニケーションの量を増やした方が良いんだろうなと思う。 そもそもコミュニケーションの量が足りていないせいで、レビュアーがそのドメインについてあまり理解が得られず、問題の表面的な部分についてのみしか発言出来ないということが沢山ある。サービスごと

    雑なレビュー - ✘╹◡╹✘
    ama-ch
    ama-ch 2014/02/07
    わかる
  • Third-Party JavaScript を読んだ - フリーフォーム フリークアウト

    移転しました http://please-sleep.cou929.nu/20121030.html

    Third-Party JavaScript を読んだ - フリーフォーム フリークアウト
  • 1