Skip to content

chore: update vscode commit #4857

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 5 commits into from
Feb 15, 2022
Merged

chore: update vscode commit #4857

merged 5 commits into from
Feb 15, 2022

Conversation

jsjoeio
Copy link
Contributor

@jsjoeio jsjoeio commented Feb 11, 2022

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

github-actions bot commented Feb 11, 2022

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

@codecov
Copy link

codecov bot commented Feb 11, 2022

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 jsjoeio temporarily deployed to CI February 14, 2022 22:47 Inactive
@jsjoeio
Copy link
Contributor Author

jsjoeio commented Feb 14, 2022

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 jsjoeio temporarily deployed to CI February 14, 2022 23:22 Inactive
@jsjoeio
Copy link
Contributor Author

jsjoeio commented Feb 14, 2022

@jsjoeio jsjoeio temporarily deployed to CI February 15, 2022 20:41 Inactive
@jsjoeio jsjoeio merged commit b26cce5 into main Feb 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 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