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

Update golangci-lint to 1.61.0 #23

Merged
merged 1 commit into from
Oct 28, 2024

Conversation

rickard-von-essen
Copy link

This resolves a problem on macOS (arm64) where lint fails with error message from typecheck.

Example:

14:02:02 [ .. ] golangci-lint
apis/accesscontextmanager/v1beta1/zz_accesslevel_terraformed.go:87:79: tr.GetName undefined (type *AccessLevel has no field or method GetName) (typecheck)
                return nil, errors.Wrapf(err, "cannot get parameters for resource '%q'", tr.GetName())
                                                                                            ^

For more information see https://golangci-lint.run/welcome/faq/#why-do-you-have-typecheck-errors

This resolves a problem on macOS (arm64) where lint fails with error
message from typecheck.

Example:

14:02:02 [ .. ] golangci-lint
apis/accesscontextmanager/v1beta1/zz_accesslevel_terraformed.go:87:79: tr.GetName undefined (type *AccessLevel has no field or method GetName) (typecheck)
                return nil, errors.Wrapf(err, "cannot get parameters for resource '%q'", tr.GetName())
                                                                                            ^

For more information see https://golangci-lint.run/welcome/faq/#why-do-you-have-typecheck-errors

Signed-off-by: Rickard von Essen <rickard.von.essen@gmail.com>
Copy link
Member

@jbw976 jbw976 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me @rickard-von-essen, I could actually use this myself in https://github.com/jbw976/conformance/tree/modernize 😉

Out of curiosity, did you see anything in the release notes, or an issue specifically talking about the mac typecheck bug? I didn't see it when quickly looking through the link you provided...

@jbw976 jbw976 merged commit d315554 into crossplane:main Oct 28, 2024
1 check passed
@rickard-von-essen
Copy link
Author

Out of curiosity, did you see anything in the release notes, or an issue specifically talking about the mac typecheck bug?

Not really, but the FAQ had as first point of resolving these issues "update to latest", and that solved it for me. 😜

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