ふだんの開発でPRを出すときに考えていること - 私が歌川です

業務の話です。OSSとかだとまた変わってくるのかもしれないし、共通することもあるかもしれません。 先に作戦を練る 実装する前に、方針段階でレビューしてもらえるとよい 自分だけでは気づけない考慮漏れとか、こういう方針もあるよっていう提案とか、いろいろ得られるものがある 先に実装完成させてから、これでは要件を満たせない・うまくいかないねってなるともったいない 巨大なPRにしない diffの大きさについては、プログラミング言語とか、利用するフレームワークによっても変わってくるので、一概には言えなさそう +1500, -1500 だけどスナップショットの更新があったとか インデント1つ下げることになったとか たとえば、あらゆる機能を1つのPRで実装してたら巨大なPRになると思う 1つのPRであらゆるものを実装しない、1機能ずつ実装するとか、1つの層だけ実装する、とか PRが巨大だと、コミュニケーシ

[iselegant]

いい知見。参考にさせていただきます。

[ikkitang1211]

よかった。 後、アウトプットをしたお陰で更に良いインプットをして頂けてるの本当にいい文化で最高、という感じになった

[rkosaka]

“PRの大きさとは何か? という問いに対して、PRが向けている関心の多さがそれにあたるのではないか” たしかにアレもやってコレもやって...というPRより、シンプルな方がいいですよね。

[mojimojikun]

えぇな

[shiba_yu36]

良い。https://shuuji3.xyz/eng-practices/review/developer/ もおすすめ

[hogashi]

よい