Skip to content
This repository has been archived by the owner on Aug 31, 2018. It is now read-only.

Protect the latest branch? #44

Open
varjmes opened this issue Sep 11, 2017 · 4 comments
Open

Protect the latest branch? #44

varjmes opened this issue Sep 11, 2017 · 4 comments

Comments

@varjmes
Copy link

varjmes commented Sep 11, 2017

I accidentally committed to master and we should be saved from ourselves by protecting the master branch. Thoughts?

@addaleax
Copy link
Contributor

We can do this, but we should always have a way to force-push to master to fix up the mistakes that we inevitably make :)

@Fishrock123
Copy link
Contributor

The reason we didn't for node boils down to two things:

  • Git is easy recoverable because the repository is fully distributed to whoever clones it. Any one of us can easily push back everything even if master is deleted or messed up entirely.
  • Being force-pushable allows some window for people to fix inevitable mistakes in e.g. metadata or anything else.

I personally do not feel that it is very necessary to do for git, or at least for any repository where a sizeable amount of people have up-to-date copies.

@ljharb
Copy link

ljharb commented Sep 11, 2017

If you use branch protection, a repo admin can disable the protection temporarily, force push, and then re-enable it, if needed.

@sandfox
Copy link

sandfox commented Sep 11, 2017

I'm ok with branch protection as a mechanism to prevent accidental pushing to master. Very much not ok as way to gatekeeper things getting landed. I think if go with the intention as @ljharb says, it's seems non to burdensome?

@addaleax addaleax changed the title Protect the master branch? Protect the latest branch? Sep 15, 2017
@addaleax addaleax changed the title Protect the latest branch? Protect the latest branch? Sep 15, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants