search

deis / logger

In-memory log buffer used by Deis Workflow.
https://deis.com
MIT License
18 stars 15 forks source link

fix(log): fix panic error when building app log message #139

Closed bacongobbler closed 7 years ago

bacongobbler commented 7 years ago

closes #136 by catching when the pod name fails the pod regex.

deis-bot commented 7 years ago

@jchauncey and @krancour are potential reviewers of this pull request based on my analysis of git blame information. Thanks @bacongobbler!

codecov-io commented 7 years ago

Codecov Report

Merging #139 into master will increase coverage by 0.63%. The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #139      +/-   ##
=========================================
+ Coverage   49.87%   50.5%   +0.63%     
=========================================
  Files          13      13              
  Lines         391     396       +5     
=========================================
+ Hits          195     200       +5     
  Misses        172     172              
  Partials       24      24
Impacted Files Coverage Δ
log/message_handler.go 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 ec3c26c...2569391. Read the comment docs.