Skip to content
This repository has been archived by the owner on Aug 21, 2022. It is now read-only.

Add option for custom error reporter #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

LinusU
Copy link
Member

@LinusU LinusU commented Aug 30, 2017

Opening for early feedback. @jonathanong I would like a way to surpress the console.error call since I've already sent the error to Sentry, and my logs follow a json format. Do you think this is a good approach?

@coveralls
Copy link

coveralls commented Aug 30, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 3d7e95e on LinusU:custom-error-reporter into df765a5 on expressjs:master.

@LinusU
Copy link
Member Author

LinusU commented Mar 15, 2018

ping @jonathanong ☺️

@LinusU
Copy link
Member Author

LinusU commented May 10, 2018

friendly ping @jonathanong ☺️

@kilhage
Copy link

kilhage commented Sep 24, 2018

+1

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants