test: create unit test for converter #351
luthfifahleviAnnotations
5 errors and 1 warning
|
golangci:
pkg/query_expr/sql_expr.go#L130
early-return: if c { ... } else { ... return } can be simplified to if !c { ... return } ... (revive)
|
|
golangci:
pkg/query_expr/es_expr_test.go#L96
S1023: redundant `return` statement (gosimple)
|
|
golangci:
pkg/query_expr/sql_expr_test.go#L96
S1023: redundant `return` statement (gosimple)
|
|
golangci:
pkg/query_expr/query_expr_test.go#L1
package should be `queryexpr_test` instead of `queryexpr` (testpackage)
|
|
golangci
issues found
|
|
golangci
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/setup-go@v4, actions/checkout@v3, golangci/golangci-lint-action@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|