Skip to content

fix:修复路由匹配失败默认转为返回全部实例 (#216) #295

fix:修复路由匹配失败默认转为返回全部实例 (#216)

fix:修复路由匹配失败默认转为返回全部实例 (#216) #295

Triggered via push August 26, 2024 04:00
Status Success
Total duration 2m 38s
Artifacts

revive.yml

on: push
Matrix: Run Revive Action
Fit to window
Zoom out
Zoom in

Annotations

66 warnings
Run Revive Action (1.16.x)
The following actions use a deprecated Node.js version 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): pkg/plugin/loadbalancer/util.go#L25
parameter 'ctx' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.16.x): pkg/plugin/loadbalancer/loadbalancer.go#L18
should have a package comment
Run Revive Action (1.16.x): test/tools/tools_vet.go#L18
should have a package comment
Run Revive Action (1.16.x): test/common/constant.go#L21
exported const ConsumerSuitServerPort should have comment (or a comment on this block) or be unexported
Run Revive Action (1.16.x): benchmark/benchmain/main.go#L46
const operationRpcDirect should be operationRPCDirect
Run Revive Action (1.16.x): test/common/constant.go#L41
exported var ServerSwitchSuitServerPort should have comment or be unexported
Run Revive Action (1.16.x): pkg/model/pb/validate.go#L79
func ConvertServerErrorToRpcError should be ConvertServerErrorToRPCError
Run Revive Action (1.16.x): plugin/loadbalancer/hash/config.go#L18
should have a package comment
Run Revive Action (1.16.x): pkg/model/pb/service.go#L428
method GetTtl should be GetTTL
Run Revive Action (1.16.x): plugin/location/local/local.go#L35
exported function New should have comment or be unexported
Run Revive Action (1.15.x)
The following actions use a deprecated Node.js version 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): benchmark/benchsimple/main.go#L18
should have a package comment
Run Revive Action (1.15.x): pkg/model/pb/metric/v2/code.go#L18
should have a package comment
Run Revive Action (1.15.x): pkg/stat/loadbalance/lb.go#L18
should have a package comment
Run Revive Action (1.15.x): benchmark/benchmain/main.go#L18
should have a package comment
Run Revive Action (1.15.x): examples/watch/instance/longpull/main.go#L18
should have a package comment
Run Revive Action (1.15.x): pkg/flow/schedule/event.go#L18
should have a package comment
Run Revive Action (1.15.x): plugin/circuitbreaker/common/half_open.go#L18
should have a package comment
Run Revive Action (1.15.x): plugin/circuitbreaker/errorrate/config.go#L18
should have a package comment
Run Revive Action (1.17.x)
The following actions use a deprecated Node.js version 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/activehealthcheck/main.go#L129
parameter 'r' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.17.x): pkg/flow/startup/client_report.go#L123
parameter 'taskKey' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.17.x): benchmark/benchmain/main.go#L325
parameter 'pos' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.17.x): pkg/flow/startup/client_report.go#L150
parameter 'event' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.17.x): pkg/flow/startup/client_report.go#L123
parameter 'taskValue' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.17.x): benchmark/benchsimple/main.go#L89
comment on exported type Durations should be of the form "Durations ..." (with optional leading article)
Run Revive Action (1.17.x): benchmark/benchmain/main.go#L325
parameter 'ctx' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.17.x): examples/circuitbreaker/provider/main.go#L70
parameter 'r' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.17.x): examples/quickstart/consumer/main.go#L58
parameter 'r' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.17.x): api_config.go#L111
comment on exported function NewConfigGroupAPIByConfig should be of the form "NewConfigGroupAPIByConfig ..."
Run Revive Action (1.18.x)
The following actions use a deprecated Node.js version 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): plugin/loadbalancer/common/half_open.go#L18
should have a package comment
Run Revive Action (1.18.x): plugin/loadbalancer/weightedrandom/wr.go#L65
parameter 'ignoreHalfOpen' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.18.x): plugin/loadbalancer/hash/config.go#L18
should have a package comment
Run Revive Action (1.18.x): test/common/constant.go#L21
exported const ConsumerSuitServerPort should have comment (or a comment on this block) or be unexported
Run Revive Action (1.18.x): plugin/loadbalancer/weightedrandom/wr.go#L18
should have a package comment
Run Revive Action (1.18.x): test/common/constant.go#L41
exported var ServerSwitchSuitServerPort should have comment or be unexported
Run Revive Action (1.18.x): plugin/location/local/local.go#L35
exported function New should have comment or be unexported
Run Revive Action (1.18.x): plugin/location/local/local.go#L18
should have a package comment
Run Revive Action (1.18.x): pkg/plugin/location/proxy.go#L26
exported type Proxy should have comment or be unexported
Run Revive Action (1.18.x): pkg/plugin/configfilter/configfilter.go#L19
should have a package comment
Run Revive Action (1.20.x)
The following actions use a deprecated Node.js version 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.20.x): pkg/model/pb/metric/code.go#L18
should have a package comment
Run Revive Action (1.20.x): api_config.go#L111
comment on exported function NewConfigGroupAPIByConfig should be of the form "NewConfigGroupAPIByConfig ..."
Run Revive Action (1.20.x): pkg/model/pb/metric/code.go#L28
exported const ExecuteSuccess should have comment (or a comment on this block) or be unexported
Run Revive Action (1.20.x): test/tools/tools_vet.go#L18
should have a package comment
Run Revive Action (1.20.x): test/common/constant.go#L18
should have a package comment
Run Revive Action (1.20.x): pkg/plugin/loadbalancer/util.go#L25
parameter 'ctx' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.20.x): pkg/plugin/configfilter/configfilter.go#L19
should have a package comment
Run Revive Action (1.20.x): pkg/plugin/location/location.go#L18
should have a package comment
Run Revive Action (1.20.x): pkg/plugin/loadbalancer/loadbalancer.go#L18
should have a package comment
Run Revive Action (1.20.x): pkg/plugin/metrics/proxy.go#L18
should have a package comment
Run Revive Action (1.19.x)
The following actions use a deprecated Node.js version 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.19.x): pkg/model/pb/metric/code.go#L28
exported const ExecuteSuccess should have comment (or a comment on this block) or be unexported
Run Revive Action (1.19.x): pkg/model/pb/metric/code.go#L18
should have a package comment
Run Revive Action (1.19.x): pkg/plugin/loadbalancer/loadbalancer.go#L18
should have a package comment
Run Revive Action (1.19.x): pkg/plugin/location/location.go#L18
should have a package comment
Run Revive Action (1.19.x): pkg/plugin/loadbalancer/util.go#L25
parameter 'ctx' seems to be unused, consider removing or renaming it as _
Run Revive Action (1.19.x): pkg/plugin/metrics/proxy.go#L18
should have a package comment
Run Revive Action (1.19.x): pkg/plugin/location/proxy.go#L26
exported type Proxy should have comment or be unexported
Run Revive Action (1.19.x): pkg/plugin/location/location.go#L27
exported const ProviderName should have comment (or a comment on this block) or be unexported
Run Revive Action (1.19.x): pkg/plugin/register/plugins.go#L21
a blank import should be only in a main or test package, or have a comment justifying it
Run Revive Action (1.19.x): pkg/plugin/register/plugins.go#L18
should have a package comment