Skip to content

Remove reindex bam from outdir#2077

Merged
FriederikeHanssen merged 5 commits intodevfrom
fix_publishing
Dec 15, 2025
Merged

Remove reindex bam from outdir#2077
FriederikeHanssen merged 5 commits intodevfrom
fix_publishing

Conversation

@FriederikeHanssen
Copy link
Copy Markdown
Contributor

reindex bam is currently published into outdir/reindex.bam

This PR disables publishing for the reindexed bam

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 11, 2025

nf-core pipelines lint overall result: Passed ✅ ⚠️

Posted for pipeline commit d49afa3

+| ✅ 224 tests passed       |+
#| ❔  13 tests were ignored |#
!| ❗   7 tests had warnings |!
Details

❗ Test warnings:

  • pipeline_todos - TODO string in base.config: Check the defaults for all processes
  • pipeline_todos - TODO string in main.nf: Optionally add in-text citation tools to this list.
  • pipeline_todos - TODO string in main.nf: Optionally add bibliographic entries to this list.
  • pipeline_todos - TODO string in main.nf: Only uncomment below if logic in toolCitationText/toolBibliographyText has been filled!
  • schema_lint - Input mimetype is missing or empty
  • schema_description - No description provided in schema for parameter: markduplicates_pixel_distance
  • schema_description - No description provided in schema for parameter: gatk_pcr_indel_model

❔ Tests ignored:

✅ Tests passed:

Run details

  • nf-core/tools version 3.5.1
  • Run at 2025-12-15 09:59:04

Base automatically changed from back-to-dev to dev December 15, 2025 09:52
Copy link
Copy Markdown

@fellen31 fellen31 left a comment

Choose a reason for hiding this comment

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

Looks good!

@FriederikeHanssen FriederikeHanssen merged commit abd035e into dev Dec 15, 2025
10 checks passed
@FriederikeHanssen FriederikeHanssen deleted the fix_publishing branch December 15, 2025 10:19
@FriederikeHanssen FriederikeHanssen mentioned this pull request Jan 18, 2026
11 tasks
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.

2 participants