Hacker Newsnew | past | comments | ask | show | jobs | submitlogin

With protected branches you can prevent force pushes to master. You would have to do a manual merge and then git push origin master to cause a problem I think.


I don't generally use feature branches on solo projects. Working directly on master (and pushing origin master at arbitrary moments to back up my work) is a deeply ingrained habit.

A code review tool that's off by default and trivial to bypass in your sleep is a poor code review tool.

This is easy to prevent in Phabricator, which will block pushes to master if they contain changes not also present in an approved unit of code review. I run into that wall a few times a week, and am reminded to move my commits into a branch.


Thanks, good point. We're extending protected branches so all code will have to undergo a code review, see https://gitlab.com/gitlab-org/gitlab-ce/issues/18193




Consider applying for YC's Winter 2026 batch! Applications are open till Nov 10

Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: