Skip to content

fix(nextflow_version): fail the build instead of publishing an empty payload#4318

Merged
mashehu merged 2 commits into
mainfrom
claude/nf-core-website-4317-byj9g0
Jul 22, 2026
Merged

fix(nextflow_version): fail the build instead of publishing an empty payload#4318
mashehu merged 2 commits into
mainfrom
claude/nf-core-website-4317-byj9g0

Conversation

@ewels

@ewels ewels commented Jul 22, 2026

Copy link
Copy Markdown
Member

The /nextflow_version endpoint is prerendered to a static file at build
time and consumed by external tooling (e.g. nf-core/setup-nextflow) that
resolves latest-* aliases from it. When the GitHub releases API was
degraded during a build, it returned no releases and the empty payload
{"latest":{},"versions":[]} got baked into the site, breaking every
downstream consumer until the next deploy (issues #3462, #4317).

Keep the endpoint prerendered (it gets heavy traffic and rarely changes),
but throw when the release list comes back empty so the build aborts and
the last successful deploy is retained rather than overwritten with a
broken one. Dropping the try/catch also lets a failed GitHub fetch
propagate and fail the build for the same reason.

Closes #4317
Closes #3462

Co-Authored-By: Claude Opus 4.8 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_01GoH7JeNdRpLFQhA8WMovWG

@netlify /nextflow_version.ts

@netlify

netlify Bot commented Jul 22, 2026

Copy link
Copy Markdown

Deploy Preview for nf-core-main-site ready!

Name Link
🔨 Latest commit 518f4a9
🔍 Latest deploy log https://app.netlify.com/projects/nf-core-main-site/deploys/6a6088c3a313740008516ff5
😎 Deploy Preview https://deploy-preview-4318--nf-core-main-site.netlify.app/nextflow_version.ts
📱 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.

@github-actions github-actions Bot deleted a comment from netlify Bot Jul 22, 2026
…payload

The /nextflow_version endpoint is prerendered to a static file at build
time and consumed by external tooling (e.g. nf-core/setup-nextflow) that
resolves latest-* aliases from it. When the GitHub releases API was
degraded during a build, it returned no releases and the empty payload
{"latest":{},"versions":[]} got baked into the site, breaking every
downstream consumer until the next deploy (issues #3462, #4317).

Keep the endpoint prerendered (it gets heavy traffic and rarely changes),
but throw when the release list comes back empty so the build aborts and
the last successful deploy is retained rather than overwritten with a
broken one. Dropping the try/catch also lets a failed GitHub fetch
propagate and fail the build for the same reason.

Closes #4317
Closes #3462

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GoH7JeNdRpLFQhA8WMovWG
@ewels
ewels force-pushed the claude/nf-core-website-4317-byj9g0 branch from f5383eb to 012e301 Compare July 22, 2026 08:13
@mashehu
mashehu merged commit 4057934 into main Jul 22, 2026
13 checks passed
@mashehu
mashehu deleted the claude/nf-core-website-4317-byj9g0 branch July 22, 2026 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants