Skip to content

chore: update vscode commit - #4857

Merged
jsjoeio merged 5 commits into
mainfrom
jsjoeio-update-code
Feb 15, 2022
Merged

chore: update vscode commit#4857
jsjoeio merged 5 commits into
mainfrom
jsjoeio-update-code

Conversation

@jsjoeio

Copy link
Copy Markdown
Contributor

This includes two fixes from coder/vscode:
- coder/vscode#43
- coder/vscode#42
@jsjoeiojsjoeio added this to the 4.0.3 milestone Feb 11, 2022
@jsjoeio
jsjoeio requested a review from a teamFebruary 11, 2022 19:55
@jsjoeiojsjoeio self-assigned this Feb 11, 2022
@jsjoeio
jsjoeio temporarily deployed to CI February 11, 2022 19:55 Inactive
@github-actions

github-actionsBot commented Feb 11, 2022

Copy link
Copy Markdown

✨ Coder.com for PR #4857 deployed! It will be updated on every commit.

@codecov

codecovBot commented Feb 11, 2022

Copy link
Copy Markdown

Codecov Report

Merging #4857 (e020f9d) into main (8fc4832) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@## main #4857 +/- ##
=======================================
Coverage 69.72% 69.72% =======================================
Files 29 29 Lines 1645 1645 Branches 363 363 =======================================
Hits 1147 1147 Misses 424 424 Partials 74 74 

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 8fc4832...e020f9d. Read the comment docs.

@jsjoeio

Copy link
Copy Markdown
ContributorAuthor

I realized why this was failing...this updates VS Code which includes a fix with two curly brackets for the proxy URL. That means we must also update the e2e test. Done and pushed

@jsjoeio

Copy link
Copy Markdown
ContributorAuthor

@jsjoeio
jsjoeio temporarily deployed to CI February 15, 2022 20:41 Inactive
@jsjoeio
jsjoeio merged commit b26cce5 into mainFeb 15, 2022
@jsjoeio
jsjoeio deleted the jsjoeio-update-code branch February 15, 2022 21:15
TinLe pushed a commit to TinLe/code-server that referenced this pull request Apr 23, 2022
* chore: update vscode commit
This includes two fixes from coder/vscode:
- coder/vscode#43
- coder/vscode#42
* fix: use double-bracket in ext. e2e test
Sign up for freeto 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

@jsjoeio@code-asher