記事へのコメント35

    • 人気コメント
    • 新着コメント
    karikari1255 karikari1255 ISMSとか上場後に承認フローどうなってるかとか言われちゃうので別で管理するのも嫌だからプルリクの承認機能使っているんだと思う。あれ嬉々としてやっている人もグループもいないというのはその通りだと思うけど

    2021/11/10 リンク

    その他
    ledsun ledsun 「コードのオーナーシップの希薄化」って悪いことだったのか…「コードの共同所有」が進んでていいことだと思ってた。

    2021/11/09 リンク

    その他
    iekusup iekusup ほー。

    2021/11/08 リンク

    その他
    ryan_aircloset ryan_aircloset そこまで固くならなくてもと思うところもありつつ、レビュイーもオーナーシップを持つべきという意見には全面同意。

    2021/11/08 リンク

    その他
    TMCNE TMCNE 環境と人の性質にかなり依る問題なので、懸念している事が実際に起こった例がないと何も話しようが無くないか

    2021/11/08 リンク

    その他
    uminomezumi uminomezumi PR作るときに自分自身(レビューイ)assignするの感覚的にいいなと思って採用してたんだけど、コードへのオーナーシップとして説明すればいいのか、なるほど

    2021/11/08 リンク

    その他
    onk onk 通らば PR、と手抜きしつつ出す場合はあって (オーナーシップの希薄化)、書いてあるようにチームに負担を強いている行為だけど、速のバランスが取れるなら今後もやっていく。とにかくレビューは関係性なんだよなー。

    2021/11/08 リンク

    その他
    inabajunmr inabajunmr GitHubのapproveボタンのテキストが変わるだけでオーナーシップが増すかというと、あまり変わらないような気はする

    2021/11/08 リンク

    その他
    ssig33 ssig33 職場の雰囲気悪すぎでは、、、

    2021/11/08 リンク

    その他
    stefafafan stefafafan まあでも実装したものがバグってた時にレビュー通ってたら、自分一人の責任ではなくチームの責任ということにしやすいのはあると思う…承認ってラベルをLGTMに変更すると良いのかな

    2021/11/08 リンク

    その他
    tg30yen tg30yen レビューの指摘の重要度で区分けすれば緩和されると思う。must/proposalをわけてproposalに対応するかはレビュイーに任せる。mustに関してはレビュアーが責任を持つ。

    2021/11/08 リンク

    その他
    inamem9999 inamem9999 承認ボタンをヨシ!ボタンにすればそれはそれで良くないか

    2021/11/08 リンク

    その他
    wata88 wata88 looks good to meでしかないのでオーナーシップについては問題意識は無いかなぁ。そもそもdiffだけでレビューできることって限られててそこがツラい

    2021/11/08 リンク

    その他
    toshiwo toshiwo 捉え方の問題じゃないかなぁ「オーナーシップの希薄化」も逆から見れば実装者の手を離れても問題ないコード(と言えるまで実装とレビューをやりきる)ということでもあるし

    2021/11/08 リンク

    その他
    katzchang katzchang 責任の希薄化はどちらの立場にもある。コードレビューで不具合を発見するのって実際難しいんだよね。

    2021/11/08 リンク

    その他
    chiroruxx chiroruxx あとでよむ

    2021/11/08 リンク

    その他
    koogawa koogawa “レビュアーの権威化” これ、とてもわかるんだよな

    2021/11/08 リンク

    その他
    ktoktokto ktoktokto 「○○人中○人の承認」方式でレビュアーをアサインした場合、経験の浅いレビュアーは積極的にレビューしないことがほとんど。権威化を防ぐという目的には寄与しないかなぁ。

    2021/11/08 リンク

    その他
    heppokopg2013 heppokopg2013 approveが必須だとオーナーシップが薄れるっていうのは他責思考すぎないかな?他の業務で考えた時、稟議が必要になったら自分の仕事じゃなくなるとか言わないし、ノーチェックでバンバン実行されるとか怖くない?

    2021/11/08 リンク

    その他
    june29 june29 Approve ナシでばんばんマージした方がうまくいくならそのチームはそれでいいと思う。自分が関わる場では「権威」や「承認」といった雰囲気を感じることはほぼないかなあ。

    2021/11/08 リンク

    その他
    hitode909 hitode909 あくまでLooks Good To Meであって、私にとっては良いと思います、第三者から見ても大丈夫そう、くらいのニュアンスでapproveしてました

    2021/11/08 リンク

    その他
    vndn vndn 未承認マージを認めた場合、レビューと切り戻しを速やかに行う必要が出る上、変更箇所が重なった場合のタルさがパないのでは。マージ後レビュー前PRの管理といったツール面も気になる。

    2021/11/08 リンク

    その他
    kagerou_ts kagerou_ts 特定の誰かにせず、メンバー全員がレビュアーで「○○人中○人」って形式にすれば権威化も薄れたりしないか。経験の浅い人も深い人もそこに混ぜる

    2021/11/08 リンク

    その他
    gfx gfx 全社的にapprove必須だけど問題はないな。そもそもコンプラ上誰もレビューしてないコードをデプロイするわけにはいかないし。レビューしてもPR主のオーナーシップは変わりませんよ、でいい気も。

    2021/11/08 リンク

    その他
    umai_bow umai_bow “承認の有無によって業務フローが左右されることでそれが権威的に扱われてしまいオーナーシップを希薄化” これらある!

    2021/11/08 リンク

    その他
    atsuououo atsuououo 組織体制による気がする。

    2021/11/08 リンク

    その他
    ipa5963 ipa5963 リング型承認と確認内容の見える化が必要。仕組みだけで質を担保しようとするのは甘え。チームに合わせて永遠に最適化を繰り返すだけ。

    2021/11/08 リンク

    その他
    revert revert 経験不足なメンバーで構成されていた所以の不幸かな。今回で学ぶだろうし強いメンターを用意できないなら多少の失敗は必要コストだよ。DevOpsな運用をしているならPRを安全側に倒すようになるから心配しなくていい

    2021/11/08 リンク

    その他
    uva uva しかしノーレビューでマージする副作用の方が大きそうだ

    2021/11/08 リンク

    その他
    teramako teramako レビューアーに権威が寄るの分かる。

    2021/11/08 リンク

    その他

    人気コメント算出アルゴリズムの一部にヤフー株式会社の「建設的コメント順位付けモデルAPI」を使用しています

    関連記事

    コードレビューとPull Request、そしてその承認機能の副作用について考える - 時計を壊せ

    用語 レビュアー 対象となるコードレビューする人のことを指します。 レビュイー レビューを受ける人...

    ブックマークしたユーザー

    • nana_kichi2021/11/18 nana_kichi
    • s_ryuuki2021/11/18 s_ryuuki
    • kyo_ago2021/11/15 kyo_ago
    • satoshi87322021/11/15 satoshi8732
    • sanko04082021/11/13 sanko0408
    • Nyoho2021/11/12 Nyoho
    • oppara2021/11/12 oppara
    • sskoji2021/11/11 sskoji
    • karikari12552021/11/10 karikari1255
    • alcus2021/11/10 alcus
    • poad10102021/11/09 poad1010
    • igaiga072021/11/09 igaiga07
    • teitei_tk2021/11/09 teitei_tk
    • sawarabi01302021/11/09 sawarabi0130
    • ledsun2021/11/09 ledsun
    • enmtknt2021/11/09 enmtknt
    • shuymn2021/11/09 shuymn
    • adwd1182021/11/08 adwd118
    すべてのユーザーの
    詳細を表示します

    同じサイトの新着

    同じサイトの新着をもっと読む

    いま人気の記事

    いま人気の記事をもっと読む

    いま人気の記事 - テクノロジー

    いま人気の記事 - テクノロジーをもっと読む

    新着記事 - テクノロジー

    新着記事 - テクノロジーをもっと読む

    KIMONO Collection -Girls- - Collection | OpenSea

    1 user https://opensea.io/

    ユニコーン 「氣志團万博2017」@袖ケ浦海浜公園 セットリスト #unicorn | K’s今日の1曲

    1 user https://ksmusic.org/