Skip to content
Back to pull request #5487

Add validation for a few string fields #5715

Sign in to view logs

Add validation for a few string fields

Add validation for a few string fields #5715

Triggered via pull request March 5, 2024 02:18
@yimincyiminc
synchronize #5487
yiminc:validate_utf8_string
Status Failure
Total duration 2m 52s
Artifacts –

golangci-lint.yml

on: pull_request
lint
2m 42s
lint
Fit to window
Zoom out
Zoom in

Annotations

4 errors and 1 warning
lint: common/searchattribute/encode_value.go#L95
err113: do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"%v is not a valid UTF-8 string\", value)" (goerr113)
lint: common/searchattribute/encode_value.go#L100
err113: do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"%v is not a valid UTF-8 string\", item)" (goerr113)
lint: common/searchattribute/encode_value_test.go#L390
SA4006: this value of `value` is never used (staticcheck)
lint
issues found
lint
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-go@v4, golangci/golangci-lint-action@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.