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

Set default branch to 'main' instead of the Nextflow default 'master' #92

Merged
merged 1 commit into from
Jun 7, 2024

Conversation

matthuska
Copy link
Collaborator

This came up in relation to changing the default branch within GitHub. I don't think that would have made a difference for Nextflow, because the hard coded default branch is 'master'. This commit changes that default for the clean pipeline to the 'main' branch.

See: https://www.nextflow.io/docs/latest/config.html#scope-manifest

@MarieLataretu
Copy link
Collaborator

Looks good to me!

(I never changed manifest.defaultBranch, e.g., in CovPipe2, which uses main instead of master and never had problems, but it makes sense for sure!)

@matthuska matthuska merged commit 36dc65f into dev Jun 7, 2024
12 checks passed
@matthuska matthuska deleted the feat/set-default-branch branch June 7, 2024 11:48
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

Successfully merging this pull request may close these issues.

2 participants