Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update node types version #33452

Merged
merged 7 commits into from Mar 28, 2022
Merged

Conversation

clavin
Copy link
Member

@clavin clavin commented Mar 25, 2022

Description of Change

Update the version of @types/node in both the repo package.json (mostly for the sake of tests, rather than scripts) as well as the published npm package.json. This doesn't match the version in DEPS because @types/node doesn't always release with Node (found this out the hard way). Fixes #33089

Release Notes

Notes: Updated node types version.

@clavin clavin requested a review from a team as a code owner March 25, 2022 23:53
@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Mar 25, 2022
@codebytere
Copy link
Member

../node_modules/@types/express-serve-static-core/index.d.ts(505,18): error TS2430: Interface 'Response' incorrectly extends interface 'ServerResponse'.
Property 'req' is optional in type 'Response' but required in type 'ServerResponse'

is failing lint

@clavin clavin force-pushed the clavin/update-node-types-version branch from 758d990 to eec60a9 Compare March 28, 2022 16:24
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Mar 28, 2022
@trop
Copy link
Contributor

trop bot commented Mar 28, 2022

@clavin has manually backported this PR to "18-x-y", please check out #33477

@VerteDinde VerteDinde merged commit 594dc7e into main Mar 28, 2022
@VerteDinde VerteDinde deleted the clavin/update-node-types-version branch March 28, 2022 21:37
@release-clerk
Copy link

release-clerk bot commented Mar 28, 2022

Release Notes Persisted

Updated node types version.

bavulapati pushed a commit to bavulapati/electron that referenced this pull request Apr 29, 2022
* chore: update node types version

* update express types to solve type conflict

* one more yarn.lock type bump

* update another types package to fix incompatible global declarations

* remove incompatible type magicks

* update our ambient types to match the node types

* fix test type
khalwa pushed a commit to solarwindscloud/electron that referenced this pull request Feb 22, 2023
* chore: update node types version

* update express types to solve type conflict

* one more yarn.lock type bump

* update another types package to fix incompatible global declarations

* remove incompatible type magicks

* update our ambient types to match the node types

* fix test type
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: Completed Stable Prep Items 🚀
Development

Successfully merging this pull request may close these issues.

[Bug]: @types/node version is outdated
3 participants