otiai10 / copy

Go copy directory recursively
https://pkg.go.dev/github.com/otiai10/copy
MIT License
722 stars 115 forks source link

Add CI tests for GopherJS and WASM build targets #79

Closed flimzy closed 2 years ago

flimzy commented 2 years ago

Follow up to https://github.com/otiai10/copy/pull/75#issuecomment-1102189321

codecov-commenter commented 2 years ago

Codecov Report

Merging #79 (db77a3c) into main (6cc6966) will not change coverage. The diff coverage is n/a.

:exclamation: Current head db77a3c differs from pull request most recent head 94e5b6c. Consider uploading reports for the commit 94e5b6c to get more accurate results

@@           Coverage Diff           @@
##             main      #79   +/-   ##
=======================================
  Coverage   77.40%   77.40%           
=======================================
  Files          13       13           
  Lines         177      177           
=======================================
  Hits          137      137           
  Misses         20       20           
  Partials       20       20           

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 6cc6966...94e5b6c. Read the comment docs.

flimzy commented 2 years ago

@otiai10 I think this PR is now ready. Sorry for all the spam I'm sure you got in your inbox, while testing this. It's hard to test CI without CI spam :shrug:

I was hoping it would be possible to make this work with both old and new versions of GopherJS, but the old versions of GopherJS are missing some vital support for tests (although I think the package works).

Whenever we release new versions of GopherJS, I'll try to make an effort to keep this package and its tests up to date.