Skip to content

chore(deps): update dependency @types/node to v16 #5170

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

Merged
merged 7 commits into from
Aug 10, 2022
Merged

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented May 6, 2022

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
@types/node ^14.17.1 -> ^16.0.0 age adoption passing confidence

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Renovate will not automatically rebase this PR, because other commits have been found.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, click this checkbox. ⚠ Warning: custom changes will be lost.

This PR has been generated by Mend Renovate. View repository job log here.

@renovate renovate bot requested a review from a team May 6, 2022 15:49
@github-actions
Copy link

github-actions bot commented May 6, 2022

✨ code-server docs for PR #5170 is ready! It will be updated on every commit.

@renovate renovate bot force-pushed the renovate/node-16.x branch 5 times, most recently from 9c550a2 to 39583cb Compare May 10, 2022 19:37
@jsjoeio
Copy link
Contributor

jsjoeio commented May 10, 2022

This will require some manual work to make sure everything is working.

@jsjoeio jsjoeio self-assigned this May 10, 2022
@jsjoeio jsjoeio added this to the May 2022 milestone May 10, 2022
@renovate renovate bot force-pushed the renovate/node-16.x branch 7 times, most recently from fbbd77e to fee0c3a Compare May 13, 2022 16:58
@renovate renovate bot force-pushed the renovate/node-16.x branch 2 times, most recently from 2f7fa6a to 6fb4c9d Compare May 20, 2022 22:25
@renovate renovate bot force-pushed the renovate/node-16.x branch 3 times, most recently from a1635d5 to 7bc2d13 Compare June 10, 2022 22:16
@renovate renovate bot force-pushed the renovate/node-16.x branch from 7bc2d13 to 9244392 Compare June 13, 2022 22:25
@renovate renovate bot force-pushed the renovate/node-16.x branch from 9244392 to b65aabc Compare June 15, 2022 15:55
@jsjoeio jsjoeio modified the milestones: June 2022, July 2022 Jun 28, 2022
@jsjoeio jsjoeio removed their assignment Jul 21, 2022
@jsjoeio jsjoeio self-assigned this Aug 8, 2022
@code-asher code-asher assigned code-asher and unassigned jsjoeio Aug 9, 2022
It seems req was `any` before so now we have to handle the types.  Also
it seems the socket is of type `stream.Duplex`, not `net.Socket`.

The ws types had to be updated to support the new type.

Unfortunately Code still uses the old type so cast for now.

In the web socket router just use a cast for the extra properties we
add.  We could add the types to the Express namespace but I am not sure
we really want these commonly accessible so keep with the casts for now.
Likely we should use Express's `locals` or something instead.
Not sure why it only just now started complaining though.
@code-asher code-asher temporarily deployed to npm August 10, 2022 19:26 Inactive
@github-actions
Copy link

✨ code-server dev build published to npm for PR #5170!

  • Last publish status: success
  • Commit: ff8264b

To install in a local project, run:

npm install @coder/code-server-pr@5170

To install globally, run:

npm install -g @coder/code-server-pr@5170

@codecov
Copy link

codecov bot commented Aug 10, 2022

Codecov Report

Merging #5170 (ff8264b) into main (7bb35af) will increase coverage by 0.01%.
The diff coverage is 75.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #5170      +/-   ##
==========================================
+ Coverage   72.42%   72.44%   +0.01%     
==========================================
  Files          30       30              
  Lines        1672     1673       +1     
  Branches      366      366              
==========================================
+ Hits         1211     1212       +1     
  Misses        398      398              
  Partials       63       63              
Impacted Files Coverage Δ
src/node/routes/vscode.ts 83.33% <0.00%> (ø)
src/node/wsRouter.ts 92.00% <83.33%> (+0.33%) ⬆️
src/node/socket.ts 89.65% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7bb35af...ff8264b. Read the comment docs.

@code-asher code-asher merged commit 91589fd into main Aug 10, 2022
@code-asher code-asher deleted the renovate/node-16.x branch August 10, 2022 21:15
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.

3 participants