Skip to content
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

Delete unused/merged branches #2077

Closed
cdrini opened this issue Apr 21, 2019 · 6 comments
Closed

Delete unused/merged branches #2077

cdrini opened this issue Apr 21, 2019 · 6 comments
Assignees
Labels
Affects: Admin/Maintenance Issues relating to support scripts, bots, cron jobs and admin web pages. [managed] Lead: @cdrini Issues overseen by Drini (Staff: Team Lead & Solr, Library Explorer, i18n) [managed] Priority: 3 Issues that we can consider at our leisure. [managed] Theme: Development Issues related to the developer experience and the dev environment. [managed] Type: Bug Something isn't working. [managed]
Projects

Comments

@cdrini
Copy link
Collaborator

cdrini commented Apr 21, 2019

Description

There are currently 144 branches. There should only really be <10. A good place to start would be deleting merged branches. This is a problem because accidentally calling git pull upstream instead of git pull upstream master will pull down ALL the branches.

Relevant url?

https://github.com/internetarchive/openlibrary/branches

Expectation

There should probably only be <10 branches, but let's close this issue when we're down to <20.

Stakeholders

@mekarpeles

@cdrini cdrini added Type: Bug Something isn't working. [managed] Type: Bug and removed Type: Bug Something isn't working. [managed] labels Apr 21, 2019
@mekarpeles
Copy link
Member

Down to 54 branches -- feel empowered to delete your own merged/closed branches, folks! Thanks @cdrini

@brad2014 brad2014 added Type: Bug Something isn't working. [managed] and removed Type: Bug labels Apr 23, 2019
@brad2014
Copy link
Collaborator

This might require someone to wrangle the branch owners in order to get them closed. @cdrini, I'm going to put this on your plate for the moment - it probably requires some research to figure out what branches are associated with what owners, yes?

@brad2014 brad2014 added Theme: Development Issues related to the developer experience and the dev environment. [managed] Affects: Admin/Maintenance Issues relating to support scripts, bots, cron jobs and admin web pages. [managed] Module: Git and removed CDrini: Dev labels May 13, 2019
@tfmorris
Copy link
Contributor

I've deleted a few more branches which were strictly behind master. Most of the remaining ones have commits by @mekarpeles which aren't fully merged, so probably need his attention to close out.

@cdrini
Copy link
Collaborator Author

cdrini commented May 14, 2019

I think most of the remaining branches are valid branches, but they shouldn't really be on the main openlibrary repo. Do you have a fork for development, @mekarpeles ? I feel like most of the remaining branches could be moved to your personal fork.

@xayhewalo xayhewalo added this to Un-Triaged in Triage Oct 18, 2019
@xayhewalo xayhewalo added Priority: 3 Issues that we can consider at our leisure. [managed] State: Backlogged labels Nov 10, 2019
@xayhewalo xayhewalo moved this from Un-Triaged to Triaged in Triage Nov 10, 2019
@xayhewalo
Copy link
Collaborator

There are still several stale branches, https://github.com/internetarchive/openlibrary/branches/stale, some of them closed. We probably need another round of deletes.

@mekarpeles mekarpeles added the Lead: @cdrini Issues overseen by Drini (Staff: Team Lead & Solr, Library Explorer, i18n) [managed] label Dec 18, 2019
@mekarpeles
Copy link
Member

Closing as not an active priority / not getting in our way / low consequence

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Affects: Admin/Maintenance Issues relating to support scripts, bots, cron jobs and admin web pages. [managed] Lead: @cdrini Issues overseen by Drini (Staff: Team Lead & Solr, Library Explorer, i18n) [managed] Priority: 3 Issues that we can consider at our leisure. [managed] Theme: Development Issues related to the developer experience and the dev environment. [managed] Type: Bug Something isn't working. [managed]
Projects
No open projects
Triage
  
Triaged
Development

No branches or pull requests

5 participants