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

fix conda and podman profile strictness #972

Merged
merged 2 commits into from
Mar 26, 2021

Conversation

phue
Copy link
Member

@phue phue commented Mar 26, 2021

In #866, we increased the strictness for container profiles by explicitly disabling other container engines if a profile is selected.

Right now nothing is broken per se, but for the conda and podman profiles, we currently do not disable charliecloud properly. This fixes it.

PR checklist

  • This comment contains a description of changes (with reason)
  • CHANGELOG.md is updated
  • If you've fixed a bug or added code that should be tested, add tests!
  • Documentation in docs is updated

@phue phue added bug Something isn't working template nf-core pipeline/component template labels Mar 26, 2021
@phue phue requested a review from jfy133 March 26, 2021 10:18
@jfy133
Copy link
Member

jfy133 commented Mar 26, 2021

Oops, nice catch @phue ! Feel free to merge!

@jfy133
Copy link
Member

jfy133 commented Mar 26, 2021

Ah missing CHANGELOG but otherwise GTG

@phue phue enabled auto-merge March 26, 2021 12:41
@phue phue merged commit 02a9abc into nf-core:dev Mar 26, 2021
@phue phue deleted the conda_podman_profile_fix branch March 26, 2021 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working template nf-core pipeline/component template
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants