Skip to content

fix starfusion download output for gtf to refflat#463

Closed
dodslaser wants to merge 5 commits intonf-core:devfrom
dodslaser:fix_starfusion_download
Closed

fix starfusion download output for gtf to refflat#463
dodslaser wants to merge 5 commits intonf-core:devfrom
dodslaser:fix_starfusion_download

Conversation

@dodslaser
Copy link
Copy Markdown

Make output.chrgtf from STARFUSION_DOWNLOAD a tuple for compatibility with GTF_TO_REFFLAT.

This fixes reference generation when --starfusion_build is false.

PR checklist

  • This comment contains a description of changes (with reason).
  • Make sure your code lints (nf-core lint).
  • Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 31, 2024

nf-core lint overall result: Failed ❌

Posted for pipeline commit 39f066c

+| ✅ 162 tests passed       |+
!| ❗   2 tests had warnings |!
-| ❌   8 tests failed       |-
Details

❌ Test failures:

  • files_exist - File must be removed: lib/Utils.groovy
  • files_exist - File must be removed: lib/WorkflowMain.groovy
  • files_exist - File must be removed: lib/NfcoreTemplate.groovy
  • files_exist - File must be removed: lib/WorkflowRnafusion.groovy
  • files_unchanged - .github/workflows/branch.yml does not match the template
  • files_unchanged - .github/workflows/linting_comment.yml does not match the template
  • files_unchanged - .github/workflows/linting.yml does not match the template
  • files_unchanged - pyproject.toml does not match the template

❗ Test warnings:

  • files_exist - File not found: conf/igenomes.config
  • pipeline_todos - TODO string in WorkflowRnafusion.groovy: Optionally add in-text citation tools to this list.

✅ Tests passed:

Run details

  • nf-core/tools version 2.13
  • Run at 2024-02-29 12:11:27

@rannick
Copy link
Copy Markdown
Collaborator

rannick commented Apr 24, 2025

Addressed already

@rannick rannick closed this Apr 24, 2025
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