Skip to content

Clean fs resources in the background - #9823

Merged
Don Jayamanne (DonJayamanne) merged 1 commit into
microsoft:masterfrom
DonJayamanne:minorFSChanges
Jan 29, 2020
Merged

Don Jayamanne (DonJayamanne) merged 1 commit into
microsoft:masterfrom
DonJayamanne:minorFSChanges

Conversation

@DonJayamanne

Copy link
Copy Markdown

For #9800
#9800 (comment)

When we've detected that a directory is r/w, we can delete the temp file in the background without blocking the call to check if dir is readonly.

@sonarqubecloud

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.9% 0.9% Duplication

@codecov-io

Codecov (codecov-io) commented Jan 29, 2020 •

Copy link
Copy Markdown

Codecov Report

Merging #9823 into master will increase coverage by <.01%.
The diff coverage is 33.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #9823      +/-   ##
==========================================
+ Coverage   60.88%   60.88%   +<.01%     
==========================================
  Files         556      556              
  Lines       30041    30042       +1     
  Branches     4554     4554              
==========================================
+ Hits        18291    18292       +1     
  Misses      10723    10723              
  Partials     1027     1027
Impacted Files Coverage Δ
src/client/common/platform/fileSystem.ts 56.78% <33.33%> (+0.21%) ⬆️

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 6e8add6...d719ffe. Read the comment docs.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@DonJayamanne
Don Jayamanne (DonJayamanne) merged commit 3b90a55 into microsoft:master Jan 29, 2020
@DonJayamanne
Don Jayamanne (DonJayamanne) deleted the minorFSChanges branch February 5, 2020 17:47
@lock lock Bot locked as resolved and limited conversation to collaborators Feb 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no-changelog No news entry required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants