nf-core / test-datasets

Test data to be used for automated testing with the nf-core pipelines
https://nf-co.re
MIT License
106 stars 354 forks source link

Very carefully delete any branches that are unnecessary (especially ones that have already been merged) #1249

Open SPPearce opened 4 months ago

SPPearce commented 4 months ago

There are currently 151 branches in the repo, some of which are not relevant (e.g. they have been created to add data but not deleted at the end of the PR).

muffato commented 2 months ago

I deleted the ones that are stale and are linked to pull requests merged or closed (most often because they were onto master and have been superseded with another one onto the right branch). Still 134 left 😩

SPPearce commented 2 months ago

Well, we have something like 110 pipelines in total including the archived ones, so that doesn't sound too bad

muffato commented 2 months ago

Oh, didn't realise there were that many ! Not too bad indeed. Here is my list of branches with a name that suggests an action could be taken.

revert-* ones branches like revert-417-modules, revert-653-melt, etc. All stale. I'm sure the number refers to a PR/issue number of an nf-core repo.

Two also probably linked to a PR/issue but have apparently no commit of their own ?

Two are very small changes that have been forgotten. Maybe they're not needed after all ? Would probably need to check that those branches are not used in any module test

There's one for sarek with a capital s: