Lowercased logrus repository dep by Albibek · Pull Request #156 · sensorbee/sensorbee · GitHub
Skip to content

Lowercased logrus repository dep#156

Merged
disktnk merged 1 commit into
sensorbee:masterfrom
Albibek:master
Aug 17, 2017
Merged

Lowercased logrus repository dep#156
disktnk merged 1 commit into
sensorbee:masterfrom
Albibek:master

Conversation

@Albibek

@Albibek Albibek commented Aug 15, 2017

Copy link
Copy Markdown

Hi.

This PR syncs @sirupsen lowercasing his logrus repository name which is now

https://github.com/sirupsen/logrus

Seeing weird case-sensitive problems? It's in the past been possible to import Logrus as both upper- and lower-case. Due to the Go package environment, this caused issues in the community and we needed a standard. Some environments experienced problems with the upper-case variant, so the lower-case was decided. Everything using logrus will need to use the lower-case: github.com/sirupsen/logrus. Any package that isn't, should be changed.

@disktnk

disktnk commented Aug 17, 2017

Copy link
Copy Markdown
Member

@disktnk

disktnk commented Aug 17, 2017

Copy link
Copy Markdown
Member

@disktnk disktnk merged commit 44bac43 into sensorbee:master Aug 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants