Skip to content

Pin nf-test in the CI instead of installing the latest release - #127

Merged
hoelzer merged 1 commit into
devfrom
ci-pin-nf-test
Sep 10, 2026
Merged

Pin nf-test in the CI instead of installing the latest release#127
hoelzer merged 1 commit into
devfrom
ci-pin-nf-test

Conversation

@hoelzer

@hoelzer hoelzer commented Aug 1, 2026

Copy link
Copy Markdown
Member

Problem

The Install nf-test step runs:

wget -qO- https://code.askimed.com/install/nf-test | bash

Without a version argument that script resolves the latest release through https://api.github.com/repos/askimed/nf-test/releases/latest — unauthenticated. On shared GitHub runners that call gets rate limited, and because the script runs under set -e the whole job dies before a single test runs:

Run wget -qO- https://code.askimed.com/install/nf-test | bash
curl: (22) The requested URL returned error: 403

This happened on the CI of #124. It is unrelated to the pipeline itself and will come back at random.

Change

Use nf-core/setup-nf-test with a pinned version, mirroring how the workflow already installs Nextflow with nf-core/setup-nextflow:

  • a pinned version means no API call to resolve "latest"
  • the action caches the download in the runner tool cache
  • GITHUB_TOKEN is set on that step so any API access it does make is authenticated

0.9.5 is the version the CI has been resolving to anyway, so nothing changes about what is tested.

Note

This touches the same workflow file as #124, but a different section (that PR only changes the NXF_VER matrix), so the two merge cleanly in either order.

The install script resolves the latest release through the unauthenticated
GitHub API, which is rate limited on shared runners. That failed a job with
`curl: (22) The requested URL returned error: 403` before a single test ran.

nf-core/setup-nf-test takes a pinned version, so no API call is needed, and
it caches the download in the tool cache. Version 0.9.5 is what the CI has
been resolving to anyway.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@matthuska

Copy link
Copy Markdown
Collaborator

Not a comment about the specific changes here (which look good), but about the action files in general: most of the actions are out of date. We can set up dependabot to automatically create PRs to bump the action versions. I can set that up for you if you like, since I'm already doing it for a few other repos.

@hoelzer

hoelzer commented Sep 10, 2026

Copy link
Copy Markdown
Member Author

Thanks — agreed, and I've opened #134 with a .github/dependabot.yml for the github-actions ecosystem: target-branch: dev so the PRs land where every other PR here does rather than on main, monthly, minor/patch bumps grouped into one PR and majors on their own so they can be tested individually.

If you'd rather set this up yourself with whatever conventions you're using in the other repos, say the word and I'll close #134 in favour of yours — no attachment to my version.

@hoelzer
hoelzer merged commit 15a272e into dev Sep 10, 2026
5 of 12 checks passed
@hoelzer
hoelzer deleted the ci-pin-nf-test branch September 10, 2026 15:29
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