Skip to content

Add strict-syntax linting pipeline#111

Open
edmundmiller wants to merge 11 commits intomainfrom
strict-syntax-health
Open

Add strict-syntax linting pipeline#111
edmundmiller wants to merge 11 commits intomainfrom
strict-syntax-health

Conversation

@edmundmiller
Copy link
Copy Markdown
Contributor

Summary

  • Add DLT pipeline to run nextflow lint on all nf-core pipelines
  • Add Evidence.dev page with charts tracking strict syntax adoption
  • Dedicated GH workflow builds Nextflow from source, runs lint daily

Files

  • .github/workflows/lint_strict_syntax.yml - Workflow with healthchecks monitoring
  • pipeline/src/nf_core_stats/strict_syntax_pipeline.py - DLT pipeline
  • pages/code/strict_syntax.md - Evidence dashboard
  • sources/nfcore_db/strict_syntax_*.sql - SQL sources

Test plan

  • Ran locally with --pipelines rnaseq - verified 90 errors captured
  • Workflow runs successfully in CI

🤖 Generated with Claude Code

edmundmiller and others added 2 commits January 13, 2026 16:28
- DLT pipeline runs nextflow lint on all nf-core pipelines
- dedicated GH Action workflow builds Nextflow edge
- Evidence page with trend charts and per-pipeline table
- replaces plotly charts with Evidence.dev visualizations

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@edmundmiller edmundmiller requested a review from a team as a code owner January 13, 2026 22:51
@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 13, 2026

Deploy Preview for nf-core-stats ready!

Name Link
🔨 Latest commit 641cb71
🔍 Latest deploy log https://app.netlify.com/projects/nf-core-stats/deploys/696be2abe62ceb00086e4cca
😎 Deploy Preview https://deploy-preview-111--nf-core-stats.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.

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Jan 13, 2026

Claude encountered an error —— View job


I'll analyze this and get back to you.

@edmundmiller edmundmiller self-assigned this Jan 13, 2026
@edmundmiller edmundmiller requested a review from ewels January 13, 2026 22:54
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Jan 13, 2026

Claude encountered an error —— View job


I'll analyze this and get back to you.

edmundmiller and others added 2 commits January 13, 2026 17:16
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
track (nextflow_sha, pipeline_sha) per pipeline; skip clone/lint if both match previous run

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Jan 13, 2026

Claude encountered an error —— View job


I'll analyze this and get back to you.

edmundmiller and others added 2 commits January 13, 2026 19:37
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
state only persists when accessed inside @dlt.resource, not @dlt.source

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Jan 14, 2026

Claude encountered an error —— View job


I'll analyze this and get back to you.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Jan 14, 2026

Claude encountered an error —— View job


I'll analyze this and get back to you.

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Jan 14, 2026

Claude encountered an error —— View job


I'll analyze this and get back to you.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@nf-core nf-core deleted a comment from claude Bot Jan 14, 2026
@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Jan 17, 2026

Claude encountered an error —— View job


I'll analyze this and get back to you.

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