search

JasonEtco / ci-reporter

🚧📝 GitHub App to show CI failure logs in PR comments
https://github.com/apps/ci-reporter
ISC License
48 stars 11 forks source link

Don't throw errors if there is no Travis match #26

Closed JasonEtco closed 6 years ago

JasonEtco commented 6 years ago

Sometimes, TravisCI will show errors that ci-reporter can't pick up on due to limitations in the Travis API. This PR adds some checks to make sure that ci-reporter has been able to find a matching error log, before crashing the app.

codecov[bot] commented 6 years ago

Codecov Report

Merging #26 into master will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #26   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           7      7           
  Lines          91     95    +4     
  Branches        7      9    +2     
=====================================
+ Hits           91     95    +4
Impacted Files Coverage Δ
src/providers/Travis.js 100% <100%> (ø) :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 39b37dc...caa1cd2. Read the comment docs.