G-Node / nix

Neuroscience information exchange format
https://readthedocs.org/projects/nixio/
Other
68 stars 36 forks source link

Back to travis for CI #784

Closed gicmo closed 5 years ago

gicmo commented 5 years ago

Github actions/workflows seem to not be ready for prime time, especially for cross PRs:

codecov[bot] commented 5 years ago

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #784   +/-   ##
======================================
  Coverage    95.2%   95.2%           
======================================
  Files         197     197           
  Lines       12205   12205           
======================================
  Hits        11620   11620           
  Misses        585     585

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 17a5b90...f015a52. Read the comment docs.

gicmo commented 5 years ago

I wonder if we should disable valgrind tests because it takes forever and I am not sure anybody is actually looking at the results.