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 existing repository branches. #693

Open
21 of 24 tasks
claymcleod opened this issue Dec 18, 2024 · 2 comments
Open
21 of 24 tasks

Clean up existing repository branches. #693

claymcleod opened this issue Dec 18, 2024 · 2 comments

Comments

@claymcleod
Copy link
Collaborator

claymcleod commented Dec 18, 2024

This issue is to triage all of the open branches (stale or active) so we can clean those up and make the repository a bit more crisp. Ideally, after this cleanup, we would require most changes to go through PRs from personal forks rather than making changes directly on this repository.

Please comment below on the branches you are designated as an owner for.

Almost Certainly Deletable

Needs Checking

Suggest Moving to a Personal Fork

Will Go Away on a Merge

Almost Certainly Needs to Stay

Other Questions

  • Do wdl-1.1.2 and wdl-1.2.1 need to be merged into wdl-1.2? They have PRs open saying they need to be merged in, but I'm not sure if this is desired based on how @jdidion has been running the branching?
@jdidion
Copy link
Collaborator

jdidion commented Jan 13, 2025

Thanks @claymcleod.

  • Everything under "Almost Certainly Deletable" can in fact be deleted
  • v1.2 can be deleted.
  • main is the legacy repository pre-WDL 1.1. It could be renamed, but should not be deleted, and renaming may break some extant links (but I think github automatically redirects renamed branches).
  • mlin/implicitly-optional-inputs: this was moved to a different branch and merged in clarify how inputs with defaults are implicitly optional #464 - can be deleted.
  • I will move those 3 of my branches to my personal fork when I have a chance.
  • wdl-1.1.2-changes can be deleted.
  • wdl-1.1.3 and wdl-1.2.1 will require merging, but those PRs are still awaiting review. Plus, Apply some fixes to WDL examples #689 needs to be merged into wdl-1.1.3 first.

@claymcleod
Copy link
Collaborator Author

Okay great. All of the above have been done, which significantly cleaned things up. That leaves two remaining items:

  • we should rename main to legacy as discussed in Slack, and
  • when you move those branches over to your personal fork, let me know.

After these, I'll call this issue completed.

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