Skip to content

ci(ci): add node 26 to test matrix#155

Merged
mcollina merged 3 commits into
mainfrom
ci/ci-26
Jul 12, 2026
Merged

ci(ci): add node 26 to test matrix#155
mcollina merged 3 commits into
mainfrom
ci/ci-26

Conversation

@Fdawgs

@Fdawgs Fdawgs commented May 18, 2026

Copy link
Copy Markdown
Member

Checklist

Signed-off-by: Frazer Smith <frazer.dev@icloud.com>

@gurgunday gurgunday left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@metcoder95

metcoder95 commented May 25, 2026

Copy link
Copy Markdown
Member

This is due nodejs/node#62083, and yargs already landed a fix in v18 but c8 still uses v17.

We can chose to wait or just apply an override for c8:

  "overrides": {
    "c8": {
      "yargs": "^18.0.0"
    }
  },

I lean on the former for it.

@Fdawgs

Fdawgs commented May 26, 2026

Copy link
Copy Markdown
Member Author

We can chose to wait or just apply an override for c8:

Happy to wait tbh rather than having to apply this override to every fastify repo and then removing it once c8 is updated.

@mcollina mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@mcollina
mcollina merged commit b93f8e2 into main Jul 12, 2026
21 checks passed
@mcollina
mcollina deleted the ci/ci-26 branch July 12, 2026 14:08
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