Template: Do not assume pipeline name is url

nf-core / tools

Python package with helper tools for the nf-core community.

https://nf-co.re

MIT License

241 stars 189 forks source link

Template: Do not assume pipeline name is url #3225

Closed maxulysse closed 1 week ago

maxulysse commented 2 weeks ago

I'd rather we use {{ name }} that way we see in the script what we have. If we have manifest.name we assume that people are using a name that org/pipeline, which should work for nf-core pipeline, but might not work for all.

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

codecov[bot] commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 75.48%. Comparing base (b834c0f) to head (f566981). Report is 14 commits behind head on dev.

Additional details and impacted files

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.