Skip to content

Fix conda environment reference#2540

Merged
Midnighter merged 6 commits into
devfrom
fix-conda
Nov 27, 2023
Merged

Fix conda environment reference#2540
Midnighter merged 6 commits into
devfrom
fix-conda

Conversation

@Midnighter

@Midnighter Midnighter commented Nov 27, 2023

Copy link
Copy Markdown
Contributor

As discussed on Slack, the current module template creates the reference to the conda environment file in a wrong manner.

PR checklist

  • This comment contains a description of changes (with reason)
  • CHANGELOG.md is updated

@mirpedrol mirpedrol left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM thank you!

@Midnighter

Copy link
Copy Markdown
Contributor Author

I'm not sure about the test failure. Can you suggest what I should change?

# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
@mashehu

mashehu commented Nov 27, 2023

Copy link
Copy Markdown
Contributor

I'm not sure about the test failure. Can you suggest what I should change?

I am looking into it. something with the linting seems to have changed

Comment thread nf_core/modules/lint/environment_yml.py Outdated
@codecov

codecov Bot commented Nov 27, 2023

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (341dbdf) 74.95% compared to head (84f92bc) 74.95%.
Report is 1 commits behind head on dev.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev    #2540   +/-   ##
=======================================
  Coverage   74.95%   74.95%           
=======================================
  Files          85       85           
  Lines        9231     9231           
=======================================
  Hits         6919     6919           
  Misses       2312     2312           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Midnighter Midnighter merged commit 3a3ecbc into dev Nov 27, 2023
@Midnighter Midnighter deleted the fix-conda branch November 27, 2023 18:53
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.

3 participants