Conversation
Some GHA variables had been introduced between the endif and the if
There was a problem hiding this comment.
Pull request overview
This PR makes small fixes to the nf-core pipeline template GitHub Actions workflows, primarily removing unnecessary “nf-core” mentions and correcting Jinja raw-block handling so GitHub Actions expressions render correctly in generated pipelines.
Changes:
- Fix Jinja
{% raw %}/{% endraw %}placement indownload_pipeline.ymlso${{ ... }}expressions aren’t emitted incorrectly. - Remove an unnecessary nf-core-specific reference in the branch protection workflow comment.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
nf_core/pipeline-template/.github/workflows/download_pipeline.yml |
Corrects raw-block boundaries and removes stray raw markers that could break rendered workflow YAML. |
nf_core/pipeline-template/.github/workflows/branch.yml |
Updates a comment to reference the templated repo name instead of nf-core. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
I think the CI error is unrelated to my change. It's complaining about module linting code. |
|
Minor template fixes to make the following PRs a (tiny) bit shorter:
download_pipeline.yml#4124 (GHA variables referenced outside of raw blocks)PR checklist
CHANGELOG.mdis updateddocsis updated