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

Clean up/cull stale branches? #1051

Open
TommyMurphyTM1234 opened this issue Apr 22, 2024 · 4 comments
Open

Clean up/cull stale branches? #1051

TommyMurphyTM1234 opened this issue Apr 22, 2024 · 4 comments

Comments

@TommyMurphyTM1234
Copy link
Collaborator

TommyMurphyTM1234 commented Apr 22, 2024

As far as I can see...

Perhaps some or all of these stale branches can be culled in order to avoid confusion - especially confusion between the master branch and the effective "master" branch riscv?

@TommyMurphyTM1234
Copy link
Collaborator Author

Can/should this be done?
Retaining so many branches that are up to 7 years old seems pointless and just confusing to me.
On the other hand maybe some of these still contain some useful changes that might be revived at some stage?
And if the branches are not of interest then they can easily be ignored?

If any/all should be deleted then maybe an admin with rights to delete them via the GitHub interface could do this rather than requiring a PR from a forked repo as it seems a bit convoluted to do it that way?

Feedback welcome...

@JanMatCodasip
Copy link
Collaborator

JanMatCodasip commented Oct 17, 2024

As for master vs. riscv -

I'd recommend to keep the current state - to retain riscv as the main branch where the development takes place. The reason is that external users of this repository already count with that (e.g. in their scripts), and I would not like to break things for them.

 

As for cleaning up old branches -

I'd also prefer to clean these up. For the time being, I'd recommend to rename all the branches to archive/<orig_name> (or similar) to clearly mark them as archived, but still leave them available should anyone need them.

@TommyMurphyTM1234 @en-sc @MarekVCodasip Would you agree?

@TommyMurphyTM1234
Copy link
Collaborator Author

Thanks for the feedback @JanMatCodasip.

As for master vs. riscv -

I'd recommend to keep the current state - to retain riscv as the main branch where the development takes place. The reason is that external users of this repository already count with that (e.g. in their scripts), and I would not like to break things for them.

I may not have been clear. riscv is the "main" branch and I was not suggesting that this should change. If anything I was suggesting that the master branch should be eliminated (or, as you suggest, marked in some way as "archived"). I was not suggesting that master be made the "main" branch or that riscv be renamed to master. (Also since the use of the term "master" is discouraged these days for other reasons).

As for cleaning up old branches -

I'd also prefer to clean these up. For the time being, I'd recommend to rename all the branches to archive/<orig_name> (or similar) to clearly mark them as archived, but still leave them available should anyone need them.

That sounds like a good compromise for now alright.
Or maybe a branch can actually be archived in GitHub? I'm not sure. Maybe it is only possible to archive a while repo?
Either way, if something like this is decided maybe it can be done via GitHub directly by an admin with suitable privileges rather than requiring a PR from a forked repo as that seems convoluted?

Thanks again for the feedback @JanMatCodasip.

@JanMatCodasip
Copy link
Collaborator

JanMatCodasip commented Oct 17, 2024

@TommyMurphyTM1234 Thank you for your reply.

If anything I was suggesting that the master branch should be eliminated (or, as you suggest, marked in some way as "archived").

Sounds good to me. We could mark the "master" as archived, too: master -> archive/master.

Or maybe a branch can actually be archived in GitHub? I'm not sure.

I don't know of an "archive" feature. We can only rename the branch (or make a tag and then delete it).

maybe it can be done via GitHub directly by an admin with suitable privileges rather than requiring a PR from a forked repo as that seems convoluted?

If all maintainers agree to do this, I'll be happy to make these changes :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants