valyala / gozstd

go wrapper for zstd
MIT License
421 stars 60 forks source link

Add support for darwin/amd64 and windows/amd64 #2

Closed zhuyie closed 6 years ago

codecov[bot] commented 6 years ago

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master      #2   +/-   ##
======================================
  Coverage    96.4%   96.4%           
======================================
  Files           5       5           
  Lines         557     557           
======================================
  Hits          537     537           
  Misses         12      12           
  Partials        8       8
Impacted Files Coverage Δ
gozstd.go 98% <ø> (ø) :arrow_up:

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 ce00aae...6020f0d. Read the comment docs.

zhuyie commented 6 years ago

Looks like Travis CI not support MS Windows right now... https://docs.travis-ci.com/user/reference/overview/#Virtualisation-Environment-vs-Operating-System

valyala commented 6 years ago

@zhuyie , thanks for the contribution . I didn't test MacOS and Windows builds, but assume it works

valyala commented 5 years ago

@zhuyie , could you re-run make clean libzstd.a under Windows and send pull request with the updated *a file?