共有
  • 記事へのコメント52

    • 注目コメント
    • 新着コメント
    その他
    オーナーコメントを固定しています
    Songmu
    オーナー Songmu blogged.

    2021/05/19 リンク

    その他
    n314
    n314 自転車置き場の議論楽しいよね。ただ前提として人数とか体制とかを先に知りたいかなあ。プログラムに詳しくないオーナーに変更点を説明してからリリースする必要があるという要件があるだけで全て変わりそうだし。

    2021/10/01 リンク

    その他
    takamii228
    takamii228 Tag(Release) Requestの実装を望む声がここにもあった。

    2021/09/06 リンク

    その他
    uxlayman
    uxlayman 賛否の前にここに上がった議論点を全部理解できる人が少ないという問題がある。Gitは無意味に複雑すぎる

    2021/06/12 リンク

    その他
    hush_in
    hush_in "「リードエンジニアにレビューしてもらってマージしてもらう」はアンチパターン"

    2021/05/21 リンク

    その他
    civitaspo
    civitaspo 粒度を図る単位に時間を使うのわかりやすくてよさそう > ブランチに1日分の作業量以上積み重ねているのにpull requestを出せなかったら黄色信号。

    2021/05/20 リンク

    その他
    tkmkg8m
    tkmkg8m “泥臭いログ含めて残っていた方が問題調査に役立つ”とか“読み手の負担を考えてコミットをきれいにするのは間違いなく良い心がけ。ただ、頑張りすぎる必要もない”とか激しく同意

    2021/05/20 リンク

    その他
    himikko
    himikko

    2021/05/19 リンク

    その他
    maruware
    maruware 全部同意。Draft pull requestはあんまりやれてない。

    2021/05/19 リンク

    その他
    turanukimaru
    turanukimaru 作りかけのコードやテストが十分でないコードを push してレビューと言うか相談していいかが悩ましい。ただ Go だと Java やってた頃に比べて設計の話をする現場が少ない。結合度を下げる!と言ってコピペする人が多い…

    2021/05/19 リンク

    その他
    for-my-internet-demo
    for-my-internet-demo あんまりrebaseに意味感じる運用してないなあ

    2021/05/19 リンク

    その他
    itinagi
    itinagi 自分のスタンスに近かった

    2021/05/19 リンク

    その他
    MonMonMon
    MonMonMon “当然自分だけが開発しているfeatureブランチに対してのみ使う 「後でamend/rebaseするかもしれないから」とfeatureブランチをpushせず手元で温められる方が嬉しくない”

    2021/05/19 リンク

    その他
    sigwyg
    sigwyg “「自分が押したマージが本番に出ていく」という体験をしてもらう” これは確かに良いかも

    2021/05/19 リンク

    その他
    hamaco
    hamaco 大体同じ感じだなー。言語化してまとめられてるのよい。

    2021/05/19 リンク

    その他
    oasis440
    oasis440 “--force-with-lease”

    2021/05/19 リンク

    その他
    k2wanko
    k2wanko ハードウェアなのかサーバーなのかスマホアプリなのかライブラリなのかとかで変わりそう

    2021/05/19 リンク

    その他
    tchssk
    tchssk “レビューワー等読み手の負担を考えてコミットをきれいにするのは間違いなく良い心がけ” スムーズにレビューして欲しかったら気配りすべきだし、そうしないならレビューに時間がかかるのを受け入れるべき

    2021/05/19 リンク

    その他
    kappaseijin
    kappaseijin レビューとPRマージで権力勾配が起きるの分かる。それぞれ得手不得手あるだろうけどいつも同じ人が指摘しているとそうなりがち

    2021/05/19 リンク

    その他
    moto_2010
    moto_2010 “モブプロをすればレビューは不要か? そうは思わない” 分かる。モブプロしてレビューなしにしたら今まで指摘された・していたものが漏れる

    2021/05/19 リンク

    その他
    tk_musik
    tk_musik モブとレビューの話同意。モブ導入時にレビュー不要をリードタイム向上根拠としてしまうと罠にハマって偉い人が個別アサインに戻す。TDDと同じで品質確保のためですの方が。スピードは絶対負けではないが状況次第。

    2021/05/19 リンク

    その他
    yimajo
    yimajo “同じ見落としをしてしまい、バグを生む可能性がある”<-同意。実際あった。コードレビューは念のための確認プロセスとして必要だったりもするけどそのプロセスが雑にやられることもあるけども

    2021/05/19 リンク

    その他
    callirhoe08
    callirhoe08 ほぼ同意。言語化に感謝です。

    2021/05/19 リンク

    その他
    hujuu
    hujuu “ブランチに1日分の作業量以上積み重ねているのにpull requestを出せなかったら黄色信号。2日分以上溜めてしまったら立ち止まったほうが良いかもしれない”

    2021/05/19 リンク

    その他
    rjge
    rjge “基本的にはプロジェクトの流儀に素直に従い、余計なストレスを抱えないのが良い” ほんこれ。オーナーシップもあるけど、最初のうちは気持ち的な慣れも必要だと思ってるのでマージは本人がやるようにしたい派

    2021/05/19 リンク

    その他
    tengo1985
    tengo1985 PRは一日の作業分、PRは出した当人がマージなど

    2021/05/19 リンク

    その他
    wyukawa
    wyukawa なるほど>“「自分が押したマージが本番に出ていく」という体験をしてもらう”

    2021/05/19 リンク

    その他
    ryuzee
    ryuzee 言語化されててありがたい

    2021/05/19 リンク

    その他
    eller
    eller GitHubにtag requestは無いですが、Release Drafterがその役割を担うものとして紹介されることありますね。

    2021/05/19 リンク

    その他
    sotet
    sotet

    2021/05/19 リンク

    その他
    damedog
    damedog 案外こういうまとまったものがない気がするので指標にしたい。

    2021/05/19 リンク

    その他

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

    アプリのスクリーンショット
    いまの話題をアプリでチェック!
    • バナー広告なし
    • ミュート機能あり
    • ダークモード搭載
    アプリをダウンロード

    関連記事

    Gitのワークフローについての私のスタンス | おそらくはそれさえも平凡な日々

    Gitのワークフロー、好みが分かれる分野で自転車置き場の議論にもなりがちだと感じている。基的にはプ...

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

    すべてのユーザーの
    詳細を表示します

    同じサイトの新着

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

    いま人気の記事

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

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

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

    新着記事 - テクノロジー

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

    同時期にブックマークされた記事