search

nextcloud / screenshots

🖼 Screenshots of Nextcloud and components
https://nextcloud.com
Creative Commons Attribution Share Alike 4.0 International
8 stars 7 forks source link

Protect master branch #12

Closed AndyScherzinger closed 8 years ago

AndyScherzinger commented 8 years ago

@mar1u5 and @jospoortvliet I just pushed and committed to master directly by accident. So I think the master should be a protected branch, enforcing PRs like we do in the source code repositories.

MorrisJobke commented 8 years ago

It is already protected, but you still can do normal pushes. Only force pushes are not allowed. Beside that we could enable LGTM bot here too and add this as status check before a PR is allowed to be merged.

AndyScherzinger commented 8 years ago

That would certainly be a solution I think.

MorrisJobke commented 8 years ago

Should work now. Please test it :)