search

polds / logrus-papertrail-hook

Papertrail hook for Logrus - https://github.com/sirupsen/logrus
MIT License
12 stars 8 forks source link

Revert to sirupsen -> Sirupsen #9

Closed evalphobia closed 7 years ago

evalphobia commented 7 years ago

As discussed on the issue https://github.com/polds/logrus-papertrail-hook/issues/8, using import path both of github.com/logrus/sirupsen and github.com/logrus/Sirupsen might occur case-insensitive import collision error...

The name changing was reverted on https://github.com/sirupsen/logrus/commit/26809363aac4cc10d49171447009ac6bccc9c655 .

I checked hooks for logrus, most of them including official Syslog hook use github.com/logrus/Sirupsen, so I would like to standardize the naming to Sirupsen to avoid the error . 💓

evalphobia commented 7 years ago

Oh, thank you for quick action 😹

polds commented 7 years ago

Thank you :)