- Use
snake_casefor branch names.- You can use
/in branch names, but it's not necessary; examples:add/money_gemfix/user_registration
- You can use
- Comment code with a single
#, write comments for humans with double examples:# logger.debug something# rubocop:disable SomeCop## This class for cool things## TODO: rewrite this
- Write good commit messages.
- Add link to Basecamp to-do, Sentry issue or to GitLab issue
in the commit description on a separated line; examples:
Resolve #...Resolve https://some-tasks.com/...
- Create one commit for some feature, fix, etc.
- You can use
git commit --amendfor this.
- You can use
- Assign to your team-lead.
- Check
Remove source branch when merged. - You can select some labels.
- Comment merge requests only when you don't expect response, otherwise create discussion.
- Resolve discussion yourself only after code changes, otherwise who opened discussion should closed it.
- Write a comment when you resolve discussion yourself:
it simplifies the tracking of MR progress and decisions for reviewers;
examples:
I did it!I did something similar to what you suggested