Skip to content
This repository has been archived by the owner on Dec 5, 2019. It is now read-only.

fix: catch worker-farm errors #380

Merged
merged 1 commit into from Dec 17, 2018
Merged

Conversation

alexander-akait
Copy link
Member

@alexander-akait alexander-akait commented Dec 17, 2018

This PR contains a:

  • bugfix
  • new feature
  • code refactor
  • test update
  • typo fix
  • metadata update

Motivation / Use-Case

Catch worker-farm errors

Breaking Changes

No

Additional Info

Refactor tests, mock os.cpus() for better testing

@alexander-akait alexander-akait force-pushed the fix-catch-worker-farm-errors branch 3 times, most recently from 52a2093 to 815e2c6 Compare December 17, 2018 11:50
@codecov
Copy link

codecov bot commented Dec 17, 2018

Codecov Report

Merging #380 into master will increase coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #380      +/-   ##
==========================================
+ Coverage   97.69%   97.71%   +0.02%     
==========================================
  Files           5        5              
  Lines         260      263       +3     
  Branches      102      102              
==========================================
+ Hits          254      257       +3     
  Misses          6        6
Impacted Files Coverage Δ
src/minify.js 100% <100%> (ø) ⬆️
src/TaskRunner.js 97.87% <100%> (+0.14%) ⬆️

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 bf36e21...4c609b5. Read the comment docs.

@alexander-akait alexander-akait force-pushed the fix-catch-worker-farm-errors branch 2 times, most recently from b4efcbd to f507d4e Compare December 17, 2018 12:23
@alexander-akait alexander-akait merged commit aa7a9fc into master Dec 17, 2018
@alexander-akait alexander-akait deleted the fix-catch-worker-farm-errors branch December 17, 2018 12:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant