evalphobia / logrus_sentry

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

case-insensitive import collision #36

Closed pawelkowalak closed 7 years ago

pawelkowalak commented 7 years ago

As per https://github.com/sirupsen/logrus/issues/451#issuecomment-264332021 logrus import path should be uppercase again.

evalphobia commented 7 years ago

@bsphere @viru Thank you guys letting me know it.

I reverted the package name and back again now. Sorry for the confusing 😭

shuLhan commented 7 years ago

And they change (and recommend) it to use lowercase again: https://github.com/sirupsen/logrus/issues/543

Can we open the issue again?