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
Updates Go to 1.21, golangci-lint to 1.55.2 #7262
Conversation
Note that, due to the change in import and init func ordering changing in Go 1.21, the grpc proxy proto codec is no longer registered after the grpc internal proto codec definition, meaning the dapr custom definition will not be registered and will cause errors. The custom proto needs to be either re-registered at runtime, or change its name in order to fix this. This is done in the gRPC object, and handler_test.go TestMain. Signed-off-by: joshvanl <me@joshvanl.dev>
See also the changes made in contrib for the linter's config. Configuration for one linter (depguard) has changed format. There are also some new linters that you may want to disable. Ideally the config should be similar in contrib and runtime: https://github.com/dapr/components-contrib/pull/3251/files#diff-6179837f7df53a6f05c522b6b7bb566d484d5465d9894fb04910dd08bb40dcc9 |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #7262 +/- ##
==========================================
- Coverage 64.55% 64.53% -0.02%
==========================================
Files 225 225
Lines 21067 21070 +3
==========================================
- Hits 13599 13597 -2
- Misses 6303 6306 +3
- Partials 1165 1167 +2 ☔ View full report in Codecov by Sentry. |
Signed-off-by: joshvanl <me@joshvanl.dev>
Signed-off-by: joshvanl <me@joshvanl.dev>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Note that, due to the change in import and init func ordering changing in Go 1.21, the grpc proxy proto codec is no longer registered after the grpc internal proto codec definition, meaning the dapr custom definition will not be registered and will cause errors. The custom proto needs to be either re-registered at runtime, or change its name in order to fix this. This is done in the gRPC object, and handler_test.go TestMain.
closes #7228