evalphobia / logrus_sentry

sentry hook for logrus
MIT License
194 stars 78 forks source link

Reverse the culprit and the message for better grouping #13

Closed belak closed 8 years ago

belak commented 8 years ago

Right now we're having multiple error messages grouped under the same heading because they use a common message. This switches the message to be the 'Culprit' so it will be grouped better, but still retain all useful information.

coveralls commented 8 years ago

Coverage Status

Coverage decreased (-0.3%) to 54.726% when pulling 060021c4a4a2cd92107a6674c272e4af3850473b on belak:belak/exception-reverse-culprit-message into 8bf132a350413d52af71dd11348365154e3703cd on evalphobia:master.

belak commented 8 years ago

I'm not sure this is actually the problem, so I'm going to drop it until I can properly diagnose what's going on over here.