refactor:调整 grpc 部分逻辑实现&补充日志打印 #144
chuntaojuntesting.yml
on: pull_request
Matrix: build
Matrix: golangci-lint
Matrix: Run Revive Action
Annotations
62 errors and 50 warnings
|
build (1.15):
logger.go#L28
exported type LogLevel should have comment or be unexported
|
|
build (1.15):
logger.go#L32
exported const LogDebug should have comment (or a comment on this block) or be unexported
|
|
build (1.15):
logger.go#L40
exported function SetLogger should have comment or be unexported
|
|
build (1.15):
logger.go#L44
exported function GetLogger should have comment or be unexported
|
|
build (1.15):
logger.go#L48
exported type Logger should have comment or be unexported
|
|
build (1.15):
ratelimit.go#L67
exported method RateLimitInterceptor.StreamServerInterceptor should have comment or be unexported
|
|
build (1.15):
resolver.go#L36
exported type ResolverContext should have comment or be unexported
|
|
build (1.15):
resolver.go#L50
comment on exported function RegisterResolverInterceptor should be of the form "RegisterResolverInterceptor ..."
|
|
build (1.15):
server_option.go#L267
comment on exported function WithPolarisRateLimit should be of the form "WithPolarisRateLimit ..."
|
|
build (1.15):
test/client_test.go#L100
should not use basic type untyped string as key in context.WithValue
|
|
build (1.18)
The job was canceled because "_1_15" failed.
|
|
build (1.18):
logger.go#L28
exported type LogLevel should have comment or be unexported
|
|
build (1.18):
logger.go#L32
exported const LogDebug should have comment (or a comment on this block) or be unexported
|
|
build (1.18):
logger.go#L40
exported function SetLogger should have comment or be unexported
|
|
build (1.18):
logger.go#L44
exported function GetLogger should have comment or be unexported
|
|
build (1.18):
logger.go#L48
exported type Logger should have comment or be unexported
|
|
build (1.18):
ratelimit.go#L67
exported method RateLimitInterceptor.StreamServerInterceptor should have comment or be unexported
|
|
build (1.18):
resolver.go#L36
exported type ResolverContext should have comment or be unexported
|
|
build (1.18):
resolver.go#L50
comment on exported function RegisterResolverInterceptor should be of the form "RegisterResolverInterceptor ..."
|
|
build (1.18):
server_option.go#L267
comment on exported function WithPolarisRateLimit should be of the form "WithPolarisRateLimit ..."
|
|
build (1.18):
test/client_test.go#L100
should not use basic type untyped string as key in context.WithValue
|
|
build (1.16)
The job was canceled because "_1_15" failed.
|
|
build (1.16):
logger.go#L28
exported type LogLevel should have comment or be unexported
|
|
build (1.16):
logger.go#L32
exported const LogDebug should have comment (or a comment on this block) or be unexported
|
|
build (1.16):
logger.go#L40
exported function SetLogger should have comment or be unexported
|
|
build (1.16):
logger.go#L44
exported function GetLogger should have comment or be unexported
|
|
build (1.16):
logger.go#L48
exported type Logger should have comment or be unexported
|
|
build (1.16):
ratelimit.go#L67
exported method RateLimitInterceptor.StreamServerInterceptor should have comment or be unexported
|
|
build (1.16):
resolver.go#L36
exported type ResolverContext should have comment or be unexported
|
|
build (1.16):
resolver.go#L50
comment on exported function RegisterResolverInterceptor should be of the form "RegisterResolverInterceptor ..."
|
|
build (1.16):
server_option.go#L267
comment on exported function WithPolarisRateLimit should be of the form "WithPolarisRateLimit ..."
|
|
build (1.16):
test/client_test.go#L100
should not use basic type untyped string as key in context.WithValue
|
|
build (1.17)
The job was canceled because "_1_15" failed.
|
|
build (1.17):
logger.go#L28
exported type LogLevel should have comment or be unexported
|
|
build (1.17):
logger.go#L32
exported const LogDebug should have comment (or a comment on this block) or be unexported
|
|
build (1.17):
logger.go#L40
exported function SetLogger should have comment or be unexported
|
|
build (1.17):
logger.go#L44
exported function GetLogger should have comment or be unexported
|
|
build (1.17):
logger.go#L48
exported type Logger should have comment or be unexported
|
|
build (1.17):
ratelimit.go#L67
exported method RateLimitInterceptor.StreamServerInterceptor should have comment or be unexported
|
|
build (1.17):
resolver.go#L36
exported type ResolverContext should have comment or be unexported
|
|
build (1.17):
resolver.go#L50
comment on exported function RegisterResolverInterceptor should be of the form "RegisterResolverInterceptor ..."
|
|
build (1.17):
server_option.go#L267
comment on exported function WithPolarisRateLimit should be of the form "WithPolarisRateLimit ..."
|
|
build (1.17):
test/client_test.go#L100
should not use basic type untyped string as key in context.WithValue
|
|
golangci-lint (1.15.x):
server.go#L76
Error return value of `srv.serverOptions.setDefault` is not checked (errcheck)
|
|
golangci-lint (1.15.x):
ratelimit.go#L67
line is 158 characters (lll)
|
|
golangci-lint (1.15.x):
resolver.go#L80
line is 123 characters (lll)
|
|
golangci-lint (1.15.x)
golangci-lint exit with code 2
|
|
golangci-lint (1.16.x)
The job was canceled because "_1_15_x" failed.
|
|
golangci-lint (1.16.x):
server.go#L76
Error return value of `srv.serverOptions.setDefault` is not checked (errcheck)
|
|
golangci-lint (1.16.x):
ratelimit.go#L67
line is 158 characters (lll)
|
|
golangci-lint (1.16.x):
resolver.go#L80
line is 123 characters (lll)
|
|
golangci-lint (1.16.x)
golangci-lint exit with code 2
|
|
golangci-lint (1.18.x)
The job was canceled because "_1_15_x" failed.
|
|
golangci-lint (1.18.x):
server.go#L76
Error return value of `srv.serverOptions.setDefault` is not checked (errcheck)
|
|
golangci-lint (1.18.x):
ratelimit.go#L67
line is 158 characters (lll)
|
|
golangci-lint (1.18.x):
resolver.go#L80
line is 123 characters (lll)
|
|
golangci-lint (1.18.x)
golangci-lint exit with code 2
|
|
golangci-lint (1.17.x)
The job was canceled because "_1_15_x" failed.
|
|
golangci-lint (1.17.x):
server.go#L76
Error return value of `srv.serverOptions.setDefault` is not checked (errcheck)
|
|
golangci-lint (1.17.x):
ratelimit.go#L67
line is 158 characters (lll)
|
|
golangci-lint (1.17.x):
resolver.go#L80
line is 123 characters (lll)
|
|
golangci-lint (1.17.x)
golangci-lint exit with code 2
|
|
Run Revive Action (1.17.x)
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/setup-go@v3, actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Run Revive Action (1.17.x):
examples/circuitbreak/provider/main.go#L42
parameter 'ctx' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.17.x):
examples/circuitbreak/provider/main.go#L18
should have a package comment
|
|
Run Revive Action (1.17.x):
examples/routing/provider/main.go#L50
parameter 'ctx' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.17.x):
examples/quickstart/consumer/main.go#L18
should have a package comment
|
|
Run Revive Action (1.17.x):
examples/ratelimit/consumer/main.go#L18
should have a package comment
|
|
Run Revive Action (1.17.x):
examples/routing/provider/main.go#L18
should have a package comment
|
|
Run Revive Action (1.17.x):
test/server_test.go#L45
parameter 'c' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.17.x):
examples/quickstart/provider/main.go#L41
parameter 'ctx' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.17.x):
test/server_test.go#L68
parameter 'ctx' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.17.x):
test/server_test.go#L61
parameter 'c' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.18.x)
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/setup-go@v3, actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Run Revive Action (1.18.x):
examples/ratelimit/provider/main.go#L39
parameter 'ctx' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.18.x):
examples/ratelimit/provider/main.go#L18
should have a package comment
|
|
Run Revive Action (1.18.x):
examples/circuitbreak/provider/main.go#L18
should have a package comment
|
|
Run Revive Action (1.18.x):
examples/ratelimit/consumer/main.go#L18
should have a package comment
|
|
Run Revive Action (1.18.x):
examples/routing/provider/main.go#L18
should have a package comment
|
|
Run Revive Action (1.18.x):
examples/circuitbreak/provider/main.go#L42
parameter 'ctx' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.18.x):
examples/routing/provider/main.go#L50
parameter 'ctx' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.18.x):
test/mock/naming.go#L18
should have a package comment
|
|
Run Revive Action (1.18.x):
examples/routing/consumer/main.go#L18
should have a package comment
|
|
Run Revive Action (1.18.x):
examples/circuitbreak/consumer/main.go#L18
should have a package comment
|
|
Run Revive Action (1.16.x)
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/setup-go@v3, actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Run Revive Action (1.16.x):
examples/circuitbreak/provider/main.go#L18
should have a package comment
|
|
Run Revive Action (1.16.x):
examples/circuitbreak/provider/main.go#L42
parameter 'ctx' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.16.x):
test/server_test.go#L45
parameter 'c' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.16.x):
examples/ratelimit/provider/main.go#L39
parameter 'ctx' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.16.x):
examples/ratelimit/consumer/main.go#L18
should have a package comment
|
|
Run Revive Action (1.16.x):
test/client_test.go#L41
parameter 'c' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.16.x):
test/server_test.go#L61
parameter 'c' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.16.x):
test/tools/tools_vet.go#L18
should have a package comment
|
|
Run Revive Action (1.16.x):
examples/ratelimit/provider/main.go#L18
should have a package comment
|
|
Run Revive Action (1.16.x):
test/server_test.go#L68
parameter 'ctx' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.15.x)
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/setup-go@v3, actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Run Revive Action (1.15.x):
examples/quickstart/provider/main.go#L41
parameter 'ctx' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.15.x):
examples/quickstart/provider/main.go#L18
should have a package comment
|
|
Run Revive Action (1.15.x):
test/server_test.go#L45
parameter 'c' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.15.x):
test/server_test.go#L61
parameter 'c' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.15.x):
test/mock/naming.go#L90
parameter 'ctx' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.15.x):
test/mock/naming.go#L122
parameter 'ctx' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.15.x):
test/server_test.go#L68
parameter 'ctx' seems to be unused, consider removing or renaming it as _
|
|
Run Revive Action (1.15.x):
examples/circuitbreak/provider/main.go#L18
should have a package comment
|
|
Run Revive Action (1.15.x):
examples/routing/consumer/main.go#L18
should have a package comment
|
|
Run Revive Action (1.15.x):
test/tools/tools_vet.go#L18
should have a package comment
|
|
build (1.15)
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, actions/setup-go@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
golangci-lint (1.15.x)
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/setup-go@v3, actions/checkout@v3, golangci/golangci-lint-action@v3.2.0. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
golangci-lint (1.15.x)
The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
golangci-lint (1.16.x)
The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
golangci-lint (1.18.x)
The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
golangci-lint (1.17.x)
The `save-state` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|