-
Notifications
You must be signed in to change notification settings - Fork 128
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
Clean up EngSys branches #3116
Comments
@danieljurek Can this be prioritized so we can close out this issue |
A lot of EngSys work needs to happen in branches off the main repo instead of in a fork. In almost every case where I create a branch from a fork I end up moving it into the main repo to test some scenario or another. djurek/* branches deleted, following up on the others... |
Following up on remaining branches:
|
No. It is still in use in docindex. link gh-pages is a one commit branch. So we need to start from an empty repo every time we push the changes. That's the reason we have to retain the gh-pages-root. |
@sima-zhu -- Looks like we should put a branch protection rule in place. |
That's great idea. I don't have permission to do so. @danieljurek Could you help setup the protection rule? |
@chidozieononiwu -- Would you have a look at the branches and let us know if we need to keep those around? #3116 (comment) |
I have removed them all. |
Thanks everyone. There are a few remaining eng/common branches for ongoing work and those will be cleaned up as that work merges. We can close this one now :) |
https://github.com/Azure/azure-sdk-for-cpp/branches/all
Some branches look like they are personal branches, some are stale. Almost all belong to EngSys, so I suggest to assign this work item to them.
I think there should not be personal or engsys branches at this point, that are only made for the purposes of making a PR. Instead, you fork the repo, create the branch there, and create a PR to this repo - just as we do with our dev branches.
If it is a feature, or another branch of this sort, intended for long life, it is ok.
The text was updated successfully, but these errors were encountered: