search

zspecza / common-tags

🔖 Useful template literal tags for dealing with strings in ES2015+
Other
1.99k stars 61 forks source link

make some changes to CI servers for (hopefully) better speed #139

Closed zspecza closed 6 years ago

zspecza commented 6 years ago

In this PR I've modified both CI servers. Travis now tests the latest stable node release, but not older releases. AppVeyor tests version 8 and version 6 on one core architecture (since I realised there are practically no differences between x86 and x64 in the context of Node.js). This significantly improved build speeds, but I also configured AppVeyor to push test coverage reports as well, and unfortunately this does not support shallow git clones, which slows down performance quite a bit. It however is still an improvement over previous iterations, but not by very much. There is one major benefit though - since Travis speeds have improved marginally, the "time-to-first-byte" so to speak is near instantaneous - it only takes around 1 minute for the result of a CI build to notify the pull request.

codecov[bot] commented 6 years ago

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #139   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          24     24           
  Lines         102    102           
  Branches       29     29           
=====================================
  Hits          102    102

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 4ea0b81...7627dd2. Read the comment docs.