Skip to content
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

cmd/compile, go/types, types2: rename 'imported and not used' error to 'imported but not used' #47089

Open
gazerro opened this issue Jul 7, 2021 · 2 comments · May be fixed by #47237
Open

cmd/compile, go/types, types2: rename 'imported and not used' error to 'imported but not used' #47089

gazerro opened this issue Jul 7, 2021 · 2 comments · May be fixed by #47237

Comments

@gazerro
Copy link
Contributor

@gazerro gazerro commented Jul 7, 2021

Commit 449b6ab has reworded, in Go 1.14, the "declared and not used" error message as "declared but not used". I propose to do the same for the "imported and not used" error message.

If an imported package is not used, instead of

imported and not used: "fmt"

the error message would be

imported but not used: "fmt"
@gopherbot gopherbot added this to the Proposal milestone Jul 7, 2021
@ianlancetaylor
Copy link
Contributor

@ianlancetaylor ianlancetaylor commented Jul 14, 2021

This is not an API change and does not need to be a proposal. Taking it out of the proposal process.

@ianlancetaylor ianlancetaylor changed the title proposal: cmd/compile, go/types, types2: rename 'imported and not used' error to 'imported but not used' cmd/compile, go/types, types2: rename 'imported and not used' error to 'imported but not used' Jul 14, 2021
@ianlancetaylor ianlancetaylor removed this from the Proposal milestone Jul 14, 2021
@ianlancetaylor ianlancetaylor added this to the Backlog milestone Jul 14, 2021
@gopherbot
Copy link

@gopherbot gopherbot commented Jul 16, 2021

Change https://golang.org/cl/334989 mentions this issue: cmd/compile, go/types, types2: rename 'imported and not used' error to 'imported but not used'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants