Repository has a large number of pointless branches

arduino / arduino-ide

Arduino IDE 2.x

https://www.arduino.cc/en/software

GNU Affero General Public License v3.0

2.25k stars 381 forks source link

Repository has a large number of pointless branches #5

Open per1234 opened 3 years ago

per1234 commented 3 years ago

This makes the project less friendly to contributors and maintainers.

ubidefeo commented 3 years ago

Time for a Purge?

kittaakos commented 3 years ago

I have cleaned up all my branches, the rest belong to someone else.

per1234 commented 3 years ago

Thanks Akos!

I even found a leftover one of mine from months ago, which I have now removed from both repos.

Strangely, the main branch shows up as one of my branches. I certainly didn't intentionally create it, and I can't think of how I would have done it accidentally, but I'm fine with deleting that one (to be created again by renaming master before the repo goes public) if nobody else needs it.

kittaakos commented 3 years ago

emoved from both repos.

I intentionally left all the branches in the old repo.

Strangely, the main branch shows up as one of my branches.

I have noticed that too. I propose merging #4 and switch to the main now. What do you think?

per1234 commented 3 years ago

I intentionally left all the branches in the old repo.

Mine was completely worthless (I just forgot to delete it after the PR was merged).

I propose merging #4 and switch to the main now. What do you think?

As long as the default is changed to main and the master branch is removed before the repo is made public, I don't care too much when it happens. However, I have had some problems with this transition in default branch names in other repos because it's easy to miss some places where master was used, and I also just have master in my muscle memory so I even end up introducing new usages of it after the transition. So it might be good to make the change sooner than later so we have time to discover any things that need to be changed to main.

kittaakos commented 3 years ago

I don't care too much when it happens.

I prefer changing it now or ASAP because:

at the moment, we do not have PRs against the master branch,
we do not have to change the base for PRs, and
we could spot issues earlier not when we want to push the go public button.

per1234 commented 3 years ago

@rsora this one is yours: https://github.com/arduino/arduino-ide/tree/rsora/nightly-changelog Is it still relevant? From this: https://github.com/bcmi-labs/arduino-editor/pull/147 it seems like no.

@sebromero this one is yours: https://github.com/arduino/arduino-ide/tree/sebromero/app-icon Is it still relevant? From this: https://github.com/bcmi-labs/arduino-editor/pull/167 it seems like no.

rsora commented 3 years ago

@rsora this one is yours: https://github.com/arduino/arduino-ide/tree/rsora/nightly-changelog Is it still relevant? From this: bcmi-labs/arduino-editor#147 it seems like no.

Remove it, we close that PR because it was stale :+1:

sebromero commented 3 years ago

@per1234 I removed it 👍

kittaakos commented 3 years ago

I will also make another clean-up cycle and remove all branches that belong to TypeFox or ex-TypeFox colleagues.

per1234 commented 3 years ago

Remove it,

OK, done. Thanks Roberto and Sebastian!

sebromero commented 3 years ago

I even found a leftover one of mine from months ago, which I have now removed from both repos.

@per1234 It's okay Per, it happens to the best ;-)

kittaakos commented 3 years ago

I will also make another clean-up cycle and remove all branches that belong to TypeFox or ex-TypeFox colleagues.

Done.