-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
module declares its path as: github.com/sirupsen/logrus but was required as: github.com/Sirupsen/logrus #1041
Comments
Anything using it should be updated to import and require If this is in another open source project I'd be happy to PR an update to it for you. Otherwise it's something you/the developers of the software will need to do. Thanks |
Unfortunately my PR #859 that could help to diagnose those issues is for now ignored. |
Add go module support Fix logrus not working for modules: sirupsen/logrus#1041 (comment)
Update from github.com/hashicorp/vault/helper/certutil to github.com/hashicorp/vault/sdk/helper/certutil"
Change from Sirupsen to sirupsen. Ref: sirupsen/logrus#1041
It seems this error is back from the dead somehow, I get this error when doing
I stumbled on this issue from pingcap while googling the error. I did look up I search the whole project for Anything changed to the library recently? |
As for github.com/opencontainers/runc, the last published version, v0.1.1, refers to Sirupsen. Try a newer version.
|
@freeformz Thanks for your kind offer. Could you provide the required PR(s?) for this error please?
|
This updates dependencies. In the process, the following changes were required: 1. logrus path has cnahed from Sirupsen/logrus -> sirupsen/logrus. Import paths have been updated to reflect this (cf. sirupsen/logrus#1041). 2. Github client requires context as first parameter. Github client usage has been updated to reflect this.
The text was updated successfully, but these errors were encountered: