Learn More
Resend Email
38
gurpreet
7y

Just got a merge request to review. It's TERRIBLE

- 93 changes
- 23 commits
- includes multiple features
- includes new project configs
- includes refactoring

NO. Please don't do that.

Do smaller commits. One feature per MR. It will help you and the reviewer :D

rant

code

js

refactor

javascript
Favorite
Ranter
Comments
  • 5
    7y
    while true; do git commit -am "something changed"; sleep 1; done
  • 3
    7y
    Oh I see bigger pull requests daily. I strongly agree that it should be limited to 1 and 1 feature only but they can get really big as some features might require a lot of changes.
  • 2
    7y
    @CodeMasterAlex well, branches are there just for that reason. I to happen to build up lots of small commits.
  • 1
    7y
    Dangggg
  • 0
    7y
    I have to do that all the time since our features are branched off master. I must commit a complete feature, else some incomplete matters suddenly go to production 😐
  • 1
    7y
    I get these kind of merge requests all the time at work. It's hell...
Related Rants
devRant © 2021 Hexical Labs LLC
Privacy Policy  |  Terms of Service
Add Comment