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

Bump snakemake pin to 7.32.3 #176

Merged
merged 1 commit into from
Sep 14, 2023
Merged

Bump snakemake pin to 7.32.3 #176

merged 1 commit into from
Sep 14, 2023

Conversation

corneliusroemer
Copy link
Member

@corneliusroemer corneliusroemer commented Sep 9, 2023

Since our our last snakemake pin bump ~6 months ago
snakemake has added support for a default profile
that allows reducing required command line flags to
a mininmum. Without default profile, one always either
needs to explicitly specify a profile or set things like
--cores all which is very annoying.

Now, you can simply put those flags in profiles/default/config.yaml
and they'll automatically be used when you invoke snakemake

7.32.3 appears to be reasonably stable, no major bugs reported in a month
I've used it in production for a month now, without issues.

Checklist:

  • Once this is agreed upon, we should mirror the bump to conda-base

Since our our last snakemake pin bump ~6 months ago
snakemake has added support for a default profile
that allows reducing required command line flags to
a mininmum. Without default profile, one always either
needs to explicitly specify a profile or set things like
`--cores all` which is very annoying.

Now, you can simply put those flags in `profiles/default/config.yaml`
and they'll automatically be used when you invoke `snakemake`

7.32.3 appears to be reasonably stable, no major bugs reported in a month
I've used it in production for a month now, without issues.
@corneliusroemer corneliusroemer requested review from tsibley and a team September 9, 2023 10:49
Copy link
Member

@victorlin victorlin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems reasonable! Thanks for looking into the new features.

@corneliusroemer
Copy link
Member Author

Actually, we're already at 7.32.3 in conda-base, and there have been no complaints, so merging

@corneliusroemer corneliusroemer merged commit a6b0577 into master Sep 14, 2023
35 checks passed
@corneliusroemer corneliusroemer deleted the update-snakemake branch September 14, 2023 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants