-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Permissions #2
Comments
Based on what you have done, I now have the ability to create new branches in the repo, but not commit directly to Must be the case that only the creator can commit / merge PRs into master? I'll do my best to keep the PRs short and sweet 😎 |
@ibillett protecting the master branch is a "best practice" it prevents, among other things, collaborators accidentally deleting the branch. Create a new feature branch with your changes and push the branch to this repo and then submit a PR. |
@ibillett Has your permissions issue been sorted? Let me know. We seemed to have lost momentum and I would like get it back :) |
@davidrpugh Yes I think so, managed to push changes to the example-auctions branch. I need to sit down and figure out security permissions before submitting a PR. Apologies for the lack of comms, it has been a crazy busy couple of weeks at work. |
@ibillett
I have re-instated write permissions for this repo (and
docker-base
) to thedocker
team. You are the maintainer for this team (which I thought should give you write permission but apparently not). I can not seem to work out how to give only one person on a team write permissions to the repos controlled by that team.The text was updated successfully, but these errors were encountered: