Skip to content

Ensure main script is the first one in the lineage WorkflowRun record#6845

Merged
bentsherman merged 3 commits intomasterfrom
fix-main-nf-lineage-workflow-run
Feb 26, 2026
Merged

Ensure main script is the first one in the lineage WorkflowRun record#6845
bentsherman merged 3 commits intomasterfrom
fix-main-nf-lineage-workflow-run

Conversation

@jorgee
Copy link
Contributor

@jorgee jorgee commented Feb 19, 2026

User on nextflow slack found a bug with how script files are listed in the lineage:
https://nextflow.slack.com/archives/C02T98A23U7/p1771421589893539

We intended for the main script to always be first, but then we sort the entire list by path, which breaks that guarantee
This PR modifies this behaviour to only sort the script files before adding the main script

Signed-off-by: jorgee <jorge.ejarque@seqera.io>
@netlify
Copy link

netlify bot commented Feb 19, 2026

Deploy Preview for nextflow-docs-staging ready!

Name Link
🔨 Latest commit 3ff02e1
🔍 Latest deploy log https://app.netlify.com/projects/nextflow-docs-staging/deploys/69a090bbe5e13300073c64e5
😎 Deploy Preview https://deploy-preview-6845--nextflow-docs-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@jorgee jorgee requested a review from bentsherman February 19, 2026 11:52
@bentsherman bentsherman merged commit ef01581 into master Feb 26, 2026
23 of 24 checks passed
@bentsherman bentsherman deleted the fix-main-nf-lineage-workflow-run branch February 26, 2026 18:38
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.

4 participants