Well done. The only piece I can disagree (maybe agree?) with, other than #10 is #6. There are a lot of great developers advocating and using "trunk-based development." The overarching theory being that merges can suck, so do them more often and smaller.
To do the practice right you would pair program so code review happens in real-time, do TDD so everything is tested, and make many small commits, any of which would be production-ready. You can use feature flags or branching by abstraction to hide functionality. If you're interested you can read more at trunkbaseddevelopment.com/. I'm not presonally doing it yet, but working on moving my team that way.
For further actions, you may consider blocking this person and/or reporting abuse
We're a place where coders share, stay up-to-date and grow their careers.
Well done. The only piece I can disagree (maybe agree?) with, other than #10 is #6. There are a lot of great developers advocating and using "trunk-based development." The overarching theory being that merges can suck, so do them more often and smaller.
To do the practice right you would pair program so code review happens in real-time, do TDD so everything is tested, and make many small commits, any of which would be production-ready. You can use feature flags or branching by abstraction to hide functionality. If you're interested you can read more at trunkbaseddevelopment.com/. I'm not presonally doing it yet, but working on moving my team that way.