Skip to content
This repository was archived by the owner on Oct 16, 2020. It is now read-only.

Conversation

@felixfbecker
Copy link
Contributor

@felixfbecker felixfbecker commented Jun 4, 2017

This removes the check for os.platform and replaces it with inspecting the incoming file path instead.

@codecov
Copy link

codecov bot commented Jun 4, 2017

Codecov Report

Merging #290 into master will increase coverage by 0.19%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #290      +/-   ##
==========================================
+ Coverage   59.41%   59.61%   +0.19%     
==========================================
  Files          14       14              
  Lines        2134     2132       -2     
  Branches      349      349              
==========================================
+ Hits         1268     1271       +3     
+ Misses        713      710       -3     
+ Partials      153      151       -2
Impacted Files Coverage Δ
src/typescript-service.ts 71.37% <ø> (ø) ⬆️
src/project-manager.ts 82.72% <100%> (+1.2%) ⬆️
src/memfs.ts 88.23% <0%> (+1.17%) ⬆️

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 4ce73bc...d92be96. Read the comment docs.

@felixfbecker felixfbecker merged commit 39c869e into master Jun 4, 2017
@felixfbecker felixfbecker deleted the os-agnostic branch June 4, 2017 16:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants