Skip to content

Conversation

@jsjoeio
Copy link
Contributor

@jsjoeio jsjoeio commented Mar 21, 2022

@jsjoeio jsjoeio added security Security related dependencies Pull requests that update a dependency file labels Mar 21, 2022
@jsjoeio jsjoeio added this to the March 2022 milestone Mar 21, 2022
@jsjoeio jsjoeio self-assigned this Mar 21, 2022
@jsjoeio jsjoeio requested a review from a team March 21, 2022 22:27
@github-actions
Copy link

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

@codecov
Copy link

codecov bot commented Mar 21, 2022

Codecov Report

Merging #5006 (48ecc8d) into main (815c4cf) will not change coverage.
The diff coverage is n/a.

❗ Current head 48ecc8d differs from pull request most recent head 0a23b29. Consider uploading reports for the commit 0a23b29 to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #5006   +/-   ##
=======================================
  Coverage   71.58%   71.58%           
=======================================
  Files          29       29           
  Lines        1675     1675           
  Branches      373      373           
=======================================
  Hits         1199     1199           
  Misses        405      405           
  Partials       71       71           

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 815c4cf...0a23b29. Read the comment docs.

@jsjoeio jsjoeio temporarily deployed to npm March 21, 2022 22:32 Inactive
Copy link
Member

@code-asher code-asher left a comment

Choose a reason for hiding this comment

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

Woah, I did not know you could replace the package entirely with resolutions

@jsjoeio jsjoeio merged commit 60ebf2f into main Mar 21, 2022
@jsjoeio jsjoeio deleted the jsjoeio/audit-fix branch March 21, 2022 23:18
TinLe pushed a commit to TinLe/code-server that referenced this pull request Apr 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file security Security related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants