Skip to content

add Node.js 26 to CI and bump test-related deps - #1148

Merged
bjohansebas merged 1 commit into
expressjs:masterfrom
krzysdz:cgitm-26-fix
Oct 8, 2026
Merged

add Node.js 26 to CI and bump test-related deps#1148
bjohansebas merged 1 commit into
expressjs:masterfrom
krzysdz:cgitm-26-fix

Conversation

@krzysdz

@krzysdz krzysdz commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Fixes nodejs/citgm#1173

Only supertest change is strictly necessary to fix this.

@socket-security

socket-security Bot commented Oct 8, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security		 Vulnerability Quality Maintenance License
Updatednpm/​nyc@​15.1.0 ⏵ 18.0.097 +110010084100
Updatednpm/​supertest@​6.3.4 ⏵ 7.3.199100100 +194 +44100
Updatednpm/​mocha@​10.8.2 ⏵ 12.0.3971009595 -1100

View full report

@krzysdz krzysdz added dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code labels Oct 8, 2026
@bjohansebas
bjohansebas merged commit 8947e16 into expressjs:master Oct 8, 2026
34 checks passed
@MikeMcC399 MikeMcC399 mentioned this pull request Oct 8, 2026
Open
@krzysdz
krzysdz deleted the cgitm-26-fix branch October 8, 2026 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bjohansebas bjohansebas bjohansebas approved these changes
@UlisesGascon UlisesGascon Awaiting requested review from UlisesGascon

Assignees

No one assigned

Labels

dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

express-session fails on Node.js 26

2 participants

@krzysdz @bjohansebas