Skip to content

Add node 26 to the test matrix.#759

Open
jdmarshall wants to merge 1 commit into
prometheus:mainfrom
jdmarshall:node26
Open

Add node 26 to the test matrix.#759
jdmarshall wants to merge 1 commit into
prometheus:mainfrom
jdmarshall:node26

Conversation

@jdmarshall

Copy link
Copy Markdown
Contributor

No description provided.

@cjihrig cjihrig left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think the CI wants a changelog entry, but happy to see this.

@jdmarshall

Copy link
Copy Markdown
Contributor Author

I think the CI wants a changelog entry, but happy to see this.

I don't think we need a changelog for every single chore, but this one might rate a mention...

@jdmarshall jdmarshall force-pushed the node26 branch 2 times, most recently from 4595cab to c98412c Compare July 2, 2026 19:16
@jdmarshall jdmarshall requested a review from cjihrig July 2, 2026 19:27

@cjihrig cjihrig left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. The only thing I'm unsure of is whether or not you want to update the engines field in package.json. Right now it has >=24, which covers this. I'm just not sure what the project's stance is. That range would also cover Node 25, for example, which may or may not be supported by the project, but is technically EOL.

Signed-off-by: Jason Marshall <jdmarshall@users.noreply.github.com>
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