DanielRuf commented 6 years ago

What kind of change is this? (Bug Fix, Feature...) Tests / CI

What is the current behavior (You can also link to an issue) CI fails because of the old Node.js versions (4 and 5 reached their EOL).

What is the new behavior this introduces (if any)

Does this introduce any breaking changes? No

Does the PR fulfill these requirements?

[ ] Tests for the changes have been added
[ ] Docs have been added or updated

Other Comments

codecov[bot] commented 6 years ago

Codecov Report

Merging #420 into master will not change coverage. The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #420   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          19     19           
  Lines         701    701           
=====================================
  Hits          701    701

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 9ebffd0...3e0caf7. Read the comment docs.

peterramsing commented 6 years ago

I can update the docs to note this. 👍

peterramsing / lost

tests: test Node.js 6, 8 and 10 #420

Codecov Report