FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

chore(lib/vscode): cleanup/update build deps by oxy · Pull Request #3314 · coder/code-server · GitHub

chore(lib/vscode): cleanup/update build deps - #3314

Merged
oxy merged 1 commit into
mainfrom
oxy/remove-electron-osx-sign
May 7, 2021
Merged

chore(lib/vscode): cleanup/update build deps#3314
oxy merged 1 commit into
mainfrom
oxy/remove-electron-osx-sign

Conversation

oxy commented May 7, 2021

Copy link
Copy Markdown

Clean up some dependencies with security issues that we never touch in our build process, and also update vsce. Tested locally, fingers crossed it all works 🤞

oxy requested a review from a team as a code owner May 7, 2021 19:32
oxy self-assigned this May 7, 2021
oxy added this to the v3.10.0 milestone May 7, 2021

codecov Bot commented May 7, 2021

Copy link
Copy Markdown

Codecov Report

Merging #3314 (c7bed92) into main (65c2510) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #3314   +/-   ##
=======================================
  Coverage   58.95%   58.95%           
=======================================
  Files          35       35           
  Lines        1703     1703           
  Branches      374      374           
=======================================
  Hits         1004     1004           
  Misses        561      561           
  Partials      138      138           

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 65c2510...c7bed92. Read the comment docs.

jsjoeio left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Working locally for me as well! LGTM!

Screen.Recording.2021-05-07.at.12.57.14.PM.mov

oxy merged commit 02a0e05 into main May 7, 2021
oxy deleted the oxy/remove-electron-osx-sign branch May 7, 2021 19:59
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL