Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add WithAppErrorHandler #7

Merged
merged 1 commit into from
Oct 15, 2017
Merged

Conversation

izumin5210
Copy link
Member

To close #5

@codecov-io
Copy link

codecov-io commented Oct 15, 2017

Codecov Report

Merging #7 into master will not change coverage.
The diff coverage is 88.88%.

Impacted file tree graph

@@          Coverage Diff           @@
##           master      #7   +/-   ##
======================================
  Coverage    82.6%   82.6%           
======================================
  Files           3       3           
  Lines          46      46           
======================================
  Hits           38      38           
  Misses          7       7           
  Partials        1       1
Impacted Files Coverage Δ
handlers.go 96.29% <88.88%> (ø) ⬆️

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 d936efa...587acbe. Read the comment docs.

@izumin5210 izumin5210 merged commit eeb4e6a into master Oct 15, 2017
@izumin5210 izumin5210 deleted the izumin5210/with-app-error-handler branch October 15, 2017 08:01
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

handler builder for errors wrapped with *apperror.Error
2 participants