search

cesium-ml / cesium

Machine Learning Time-Series Platform
Other
670 stars 101 forks source link

Only deploy docs when on master #238

Closed stefanv closed 7 years ago

stefanv commented 7 years ago

It looks like these checks were removed with the transition to doctr. It did not cause any problems, because the various branches do not have the tokens required to push. Still, for safety, add a check to ensure deploying only from the master branch, and not from a PR.

codecov-io commented 7 years ago

Codecov Report

Merging #238 into master will not impact coverage.

@@           Coverage Diff           @@
##           master     #238   +/-   ##
=======================================
  Coverage   94.06%   94.06%           
=======================================
  Files          43       43           
  Lines        2191     2191           
  Branches      272      272           
=======================================
  Hits         2061     2061           
  Misses         73       73           
  Partials       57       57

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 9a7eb12...924079c. Read the comment docs.

bnaul commented 7 years ago

Doubles as a free codecov test! :+1:

stefanv commented 7 years ago

Nice, codecov works again!