Codecov Report

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

@@           Coverage Diff           @@
##           master      #39   +/-   ##
=======================================
  Coverage   83.33%   83.33%           
=======================================
  Files           2        2           
  Lines          48       48           
=======================================
  Hits           40       40           
  Misses          7        7           
  Partials        1        1

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 1951ba8...2dfd520. Read the comment docs.

rmax commented 4 years ago

Integration tests failed due to GH api rate limit :(

mszostok commented 4 years ago

Hi @rmax Thanks for your PR! 🙂

I'm checking that issue and will merge your pr once I will resolve it ;)

mszostok commented 4 years ago

ok, the problem is related to Travis itself:

Encrypted environment variables have been removed for security reasons.

so token was removed from the integration tests.

https://docs.travis-ci.com/user/pull-requests#pull-requests-and-security-restrictions

I will work on that separately. Anyway thanks for your PR :)

mszostok / codeowners-validator

fix: add spacing around pipe operator #39

Codecov Report