abramovic/logrus_influxdb

Import path should be "github.com/sirupsen/logrus", not "github.com/Sirupsen/logrus"

KateGo520 opened this issue · 1 comments

Background

I find that github.com/sirupsen/logrus and github.com/Sirupsen/logrus coexist in this repo:
Gopkg.toml

[[constraint]]
  name = "github.com/Sirupsen/logrus"
  version = "1.4.2"

[[constraint]]
  branch = "master"
  name = "github.com/influxdata/influxdb1-client"

[[constraint]]
  name = "github.com/sirupsen/logrus"
  version = "1.4.2"

Some places used github.com/Sirupsen/logrus. Some places used github.com/sirupsen/logrus. But github.com/sirupsen/logrus and github.com/Sirupsen/logrus are the same repo.
As README of logrus said, github.com/Sirupsen/logrus is the old path, anything using it should be updated to import and require github.com/sirupsen/logrus.

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.

Solution

Replace all the import paths, change "github.com/Sirupsen/logrus" to "github.com/sirupsen/logrus".
Where did you import it: https://github.com/abramovic/logrus_influxdb/search?q=github.com%2FSirupsen%2Flogrus&unscoped_q=github.com%2FSirupsen%2Flogrus

@abramovic @davidq2q Could you help me review this issue? Thx :p