CI/Linters Need Some Love #20218
Labels
enhancement
Requests to existing resources that expand the functionality or scope.
linter
Pertains to changes to or issues with the various linters.
technical-debt
Addresses areas of the codebase that need refactoring or redesign.
Community Note
Description
The role CI plays in reducing risk and improving code quality cannot be overstated. We need more and continuing checks. However, we have (at least) three problems that would be excellent to address:
.semgrep.yml
is getting bloated - semgrep is fast and is our preferred way going forward for analysis. However, the config file needs to be managed. We could break it into multiple files. We need to start checking off exclusions by making fixes to clean up the file.Acceptance Test Linting / validate-terraform
is slow - This check often takes over 30 minutes. I suspect we can get the same benefit with a faster approach (semgrep?). (Acceptance Test Linting / validate-terraform (pull_request) Successful in 31m
)References
The text was updated successfully, but these errors were encountered: