Update overrides section behavior in imports and inline
#4646
aknyshcodeql.yml
on: pull_request
PR Size Labeler
11s
Matrix: Analyze
Annotations
21 errors
|
[golangci] internal/exec/stack_processor_utils.go#L551:
internal/exec/stack_processor_utils.go#L551
cognitive complexity 729 of func `ProcessStackConfig` is high (> 20) (gocognit)
|
|
[golangci] internal/exec/vendor_utils.go#L141:
internal/exec/vendor_utils.go#L141
cognitive complexity 41 of func `ReadAndProcessVendorConfigFile` is high (> 20) (gocognit)
|
|
[golangci] internal/exec/stack_processor_utils.go#L225:
internal/exec/stack_processor_utils.go#L225
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"invalid stack manifest '%s'\\n%v%s\", relativeFilePath, err, stackManifestTemplatesErrorMessage)" (err113)
|
|
[golangci] internal/exec/stack_processor_utils.go#L235:
internal/exec/stack_processor_utils.go#L235
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"invalid stack manifest '%s'\\n%v%s\", relativeFilePath, err, stackManifestTemplatesErrorMessage)" (err113)
|
|
[golangci] internal/exec/stack_processor_utils.go#L292:
internal/exec/stack_processor_utils.go#L292
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"invalid 'overrides' section in the stack manifest '%s'\", relativeFilePath)" (err113)
|
|
[golangci] internal/exec/stack_processor_utils.go#L299:
internal/exec/stack_processor_utils.go#L299
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"invalid 'terraform' section in the stack manifest '%s'\", relativeFilePath)" (err113)
|
|
[golangci] internal/exec/stack_processor_utils.go#L304:
internal/exec/stack_processor_utils.go#L304
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"invalid 'terraform.overrides' section in the stack manifest '%s'\", relativeFilePath)" (err113)
|
|
[golangci] internal/exec/stack_processor_utils.go#L312:
internal/exec/stack_processor_utils.go#L312
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"invalid 'helmfile' section in the stack manifest '%s'\", relativeFilePath)" (err113)
|
|
[golangci] internal/exec/stack_processor_utils.go#L317:
internal/exec/stack_processor_utils.go#L317
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"invalid 'terraform.overrides' section in the stack manifest '%s'\", relativeFilePath)" (err113)
|
|
[golangci] internal/exec/stack_processor_utils.go#L348:
internal/exec/stack_processor_utils.go#L348
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"invalid empty import in the manifest '%s'\", relativeFilePath)" (err113)
|
|
[golangci] internal/exec/stack_processor_utils.go#L540:
internal/exec/stack_processor_utils.go#L540
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"ProcessYAMLConfigFile: Merge: Deep-merge the stack manifest and all the imports: Error: %v\", err)" (err113)
|
|
[golangci] internal/exec/vendor_utils.go#L136:
internal/exec/vendor_utils.go#L136
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"to vendor a component, the '--component' (shorthand '-c') flag needs to be specified.\\n\"+\n\t\"Example: atmos vendor pull -c <component>\\n%s\", q)" (err113)
|
|
[golangci] internal/exec/workflow.go#L128:
internal/exec/workflow.go#L128
do not define dynamic errors, use wrapped static errors instead: "fmt.Errorf(\"%s\", errorMarkdown)" (err113)
|
|
[golangci] internal/exec/stack_processor_utils.go#L552:
internal/exec/stack_processor_utils.go#L552
hugeParam: atmosConfig is heavy (5760 bytes); consider passing it by pointer (gocritic)
|
|
[golangci] internal/exec/vendor_utils.go#L142:
internal/exec/vendor_utils.go#L142
hugeParam: atmosConfig is heavy (5760 bytes); consider passing it by pointer (gocritic)
|
|
[golangci] pkg/utils/markdown_utils.go#L20:
pkg/utils/markdown_utils.go#L20
Comment should end in a period (godot)
|
|
[golangci] pkg/utils/markdown_utils.go#L12:
pkg/utils/markdown_utils.go#L12
import "github.com/charmbracelet/log" imported as "l" but must be "log" according to config (importas)
|
|
[golangci] internal/exec/stack_processor_utils.go#L297:
internal/exec/stack_processor_utils.go#L297
`if ok` has complex nested blocks (complexity: 4) (nestif)
|
|
[golangci] internal/exec/stack_processor_utils.go#L310:
internal/exec/stack_processor_utils.go#L310
`if ok` has complex nested blocks (complexity: 4) (nestif)
|
|
[golangci] internal/exec/stack_processor_utils.go#L507:
internal/exec/stack_processor_utils.go#L507
`if len(finalTerraformOverrides) > 0 || len(finalHelmfileOverrides) > 0` has complex nested blocks (complexity: 19) (nestif)
|
|
[golangci] internal/exec/stack_processor_utils.go#L1926:
internal/exec/stack_processor_utils.go#L1926
`if !isDirectory && isYaml` has complex nested blocks (complexity: 7) (nestif)
|