Skip to content

Conversation

@matthewrobertson
Copy link
Member

@matthewrobertson matthewrobertson commented Dec 10, 2024

Updated all dependencies and fixed resulting broken tests.

Unfortunately, we are still stuck on some older versions of packages until we completely drop support for nodejs10 and nodejs12 in 2025.

BEGIN_COMMIT_OVERRIDE
deps: update all dependencies
END_COMMIT_OVERRIDE

@matthewrobertson matthewrobertson force-pushed the update-deps branch 10 times, most recently from ec138ba to b065429 Compare December 11, 2024 22:54
@matthewrobertson matthewrobertson merged commit 03b3577 into main Dec 11, 2024
56 checks passed
@matthewrobertson matthewrobertson deleted the update-deps branch December 11, 2024 23:25
matthewrobertson added a commit that referenced this pull request Dec 12, 2024
@matthewrobertson matthewrobertson changed the title chore(deps): upgrade all dependencies deps: upgrade all dependencies Dec 12, 2024
},
"dependencies": {
"@types/express": "4.17.21",
"@types/express": "^5.0.0",
Copy link

Choose a reason for hiding this comment

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

I see Express v4 referenced in dependencies, so the v5 types here are causing compatibility issues when used in other libraries. See issue #673.

Should types also be moved to devDependencies?

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