Skip to content
reviewdog

feat(ruleset): add dynamic evaluation with expr #6574

Sign in to view logs

feat(ruleset): add dynamic evaluation with expr

feat(ruleset): add dynamic evaluation with expr #6574

Triggered via pull request December 31, 2024 14:33
@JordanSussmanJordanSussman
opened #1233
eval-ruleset
Status Failure
Total duration 2m 37s
Artifacts –

reviewdog.yml

on: pull_request
diff-review
2m 26s
diff-review
full-review
2m 27s
full-review
Fit to window
Zoom out
Zoom in

Annotations

16 errors and 2 warnings
diff-review: compiler/native/expand.go#L396
string `golang` has 3 occurrences, make it a constant (goconst)
diff-review: compiler/types/pipeline/ruleset.go#L26
tag is not aligned, should be: json:"eval,omitempty" yaml:"eval,omitempty" (tagalign)
diff-review: compiler/types/pipeline/ruleset.go#L106
non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
diff-review: compiler/types/pipeline/ruleset.go#L111
non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
diff-review: compiler/types/pipeline/ruleset.go#L26
tag is not aligned, should be: json:"eval,omitempty" yaml:"eval,omitempty" (tagalign)
diff-review: compiler/types/pipeline/ruleset.go#L106
non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
diff-review: compiler/types/pipeline/ruleset.go#L111
non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
diff-review
reviewdog exited with status code: 1
full-review: compiler/native/expand.go#L396
string `golang` has 3 occurrences, make it a constant (goconst)
full-review: compiler/types/pipeline/ruleset.go#L26
tag is not aligned, should be: json:"eval,omitempty" yaml:"eval,omitempty" (tagalign)
full-review: compiler/types/pipeline/ruleset.go#L106
non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
full-review: compiler/types/pipeline/ruleset.go#L111
non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
full-review: compiler/native/expand.go#L396
string `golang` has 3 occurrences, make it a constant (goconst)
full-review: compiler/types/pipeline/ruleset.go#L26
tag is not aligned, should be: json:"eval,omitempty" yaml:"eval,omitempty" (tagalign)
full-review: compiler/types/pipeline/ruleset.go#L106
non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
full-review: compiler/types/pipeline/ruleset.go#L111
non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint)
diff-review
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
full-review
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636