search

evalphobia / logrus_sentry

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

Update casing of logrus package #35

Closed connor4312 closed 7 years ago

connor4312 commented 7 years ago

See https://github.com/sirupsen/logrus/issues/451, the username is now lowercase. As-is this readily causes failing builds on case-sensitive systems (linux/osx) 😉

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 82.759% when pulling 574ad3886db77b22e55ed1972bf4f1b327b13837 on WatchBeam:master into 02fca77698040f9b436f5caec92ac43119c88bde on evalphobia:master.

evalphobia commented 7 years ago

Oh! he finally changed his name 😮 Thanks!

bsphere commented 7 years ago

the change was reverted back, now logrus_sentry is the only package that imports "github.com/sirupsen/logrus" and is the one causing failed builds.

https://github.com/sirupsen/logrus/commit/26809363aac4cc10d49171447009ac6bccc9c655