prmr / Creco

Recommendation System for Consumer Products
Apache License 2.0
6 stars 2 forks source link

Clean dead branches #77

Closed prmr closed 10 years ago

prmr commented 10 years ago

The last person who did work on a branch that can safely be deleted should delete the branch. For merged branches this is a no-brainer. For unmerged branches, think about it carefully, and talk to anyone else working on the branch.

enewe101 commented 10 years ago

Yessss! Seriously. Remember that you have to delete the local version, and then also delete the remote version. This looks like

git branch -D <branch-name>
git push origin :<branch-name>

The second line is actually pushing nothing into remote's version of the branch, which is how you delete the remote's copy of the branch.

prmr commented 10 years ago

Here's how to do it with eGit: First delete your local branch:

  1. Team | Advanced | Delete Branch...
  2. Pick a local branch and OK Don't try to do this with a remote branch. To remove a remote branch:
  3. Team | Remote | Push ...
  4. (assuming your repo is selected) Next
  5. Pick the branch in remove ref to delete.
  6. Click add spec
  7. Click Finish. This should remove the branch on the origin repo as well.
forgues commented 10 years ago

This can also be done through the GitHub interface.

  1. Go to the Code section.
  2. Click on branches (between commits and releases, at the top of the page).
  3. This page shows all branches that have been merged into master, with the option to delete each of them.
  4. You can also click "view unmerged branches" to see the other branches, and delete those if needed.
prmr commented 10 years ago

See the updated comment on who should delete branches.

enewe101 commented 10 years ago

Please address your dead branches:

  @forgues remotes/origin/AttributeCorrelator
  @nishanth1991 remotes/origin/Autocomplete_revised
  @MariamN remotes/origin/Integrate_RankingAlgo_AttrCorrelation
  @prmr remotes/origin/Issue0022
  @asutcl remotes/origin/Issue0045
  @prmr remotes/origin/Issue0050
  @ceipher remotes/origin/Issue0068
  @prmr remotes/origin/Issue0069
  @prmr remotes/origin/Issue0075-mpr
  @asutcl remotes/origin/Issue0084
  @MariamN remotes/origin/Issue31
  @asutcl remotes/origin/Issue72
  @forgues remotes/origin/Issue75_RefactorProductRanker
  @priyasidhaye remotes/origin/Logging
  @nishanth1991 remotes/origin/ProductFilter
  @mangalagb remotes/origin/Productnames
  @MariamN remotes/origin/RankingExplanation_issue79
  @asutcl remotes/origin/attribue_merge2
  @asutcl remotes/origin/attribute_merge
  @mangalagb remotes/origin/cleanupProductSearch
  @asutcl remotes/origin/feature_extraction
  @MariamN remotes/origin/features_display
  @MariamN remotes/origin/issue67
  @enewe101 remotes/origin/persistance_startup // I'm not sure what to do with this non-integrated but useful code
  @MariamN remotes/origin/rankingAlgorithm_issue47
  @mangalagb remotes/origin/removeProductSearch
  @asutcl remotes/origin/testing_data
  @mangalagb remotes/origin/typedValue
  @priyasidhaye remotes/origin/typedValue2
  @MariamN remotes/origin/update_product_ranks