-
Notifications
You must be signed in to change notification settings - Fork 339
Request: Remove dependency linters for go.mod #98
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
Comments
cli/pkg imports
github.com/manifoldco/promptui imports
github.com/alecthomas/gometalinter imports
gopkg.in/alecthomas/kingpin.v3-unstable imports
github.com/nicksnyder/go-i18n/i18n: module github.com/nicksnyder/go-i18n@latest (v2.0.2+incompatible) found, but does not contain package github.com/nicksnyder/go-i18n/i18n Although it might be related to this: golang/go#34085 |
Just ran into the same problem as adnaan. |
Can't get why this issues still not fixed? |
Bump. This is also happening in latest. https://github.com/alecthomas/gometalinter is now deprecated and they are encouraging users to use https://github.com/golangci/golangci-lint That said having it in the dependencies seems problematic for consuming projects. @royeo @LuizBranco @Minivera or others, could we have the dependency removed? |
I was able to fix the issue with go-i18n by adding the following to my go.mod:
|
@mschneider82 thanks that also worked for me. Would be great to get it really "fixed" but this allows me to run |
Uh oh!
There was an error while loading. Please reload this page.
If I do not need the linter dependency to run promptui, can I remove it from go.mod?
Dependencies of tools using promptui also include linter, which can not be excluded.
I do not want to include dependencies in go.sum that are unnecessary for execution.
It would be greatly appreciated if you could explain the another solution.
go.sum
The text was updated successfully, but these errors were encountered: