Skip to content

fix: ext-host editors error on closing diff editors#261

Merged
erha19 merged 1 commit into
v2.13.xfrom
fix/ext-host-editors-sync-failue-patch
Dec 30, 2021
Merged

fix: ext-host editors error on closing diff editors#261
erha19 merged 1 commit into
v2.13.xfrom
fix/ext-host-editors-sync-failue-patch

Conversation

@erha19
Copy link
Copy Markdown
Member

@erha19 erha19 commented Dec 30, 2021

Types

  • 🐛 Bug Fixes

Background or solution

Changelog

fix ext-host editors error on closing diff editors

Copy link
Copy Markdown
Member

@hacke2 hacke2 left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #261 (741cf19) into v2.13.x (56d9874) will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           v2.13.x     #261      +/-   ##
===========================================
- Coverage    59.40%   59.40%   -0.01%     
===========================================
  Files         1180     1180              
  Lines        72338    72338              
  Branches     14972    14972              
===========================================
- Hits         42970    42969       -1     
- Misses       26764    26765       +1     
  Partials      2604     2604              
Impacted Files Coverage Δ
...nsion/src/browser/vscode/api/main.thread.editor.ts 44.83% <ø> (ø)
...erences/src/browser/preference-settings.service.ts 79.16% <0.00%> (-0.70%) ⬇️

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 423ea3a...741cf19. Read the comment docs.

@erha19 erha19 merged commit e0ebe4a into v2.13.x Dec 30, 2021
@erha19 erha19 deleted the fix/ext-host-editors-sync-failue-patch branch December 30, 2021 12:22
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