Skip to content

Conversation

@atyu1
Copy link
Contributor

@atyu1 atyu1 commented Jun 4, 2025

Description

Add support for:

  • Helm charts linting
  • Helm Readme generation

Helm chart linting is universal and automatically checks all directories inside, if presents in repo which are called: charts
Helm Readme generate README

Both does not fail, if repo does not have custom chart.

New tools added to .tools-version to support it

Type of change

  • A bug fix (PR prefix fix)
  • A new feature (PR prefix feat)
  • A code change that neither fixes a bug nor adds a feature (PR prefix refactor)
  • Adding missing tests or correcting existing tests (PR prefix test)
  • Changes that do not affect the meaning of the code like white-spaces, formatting, missing semi-colons, etc. (PR prefix style)
  • Changes to our CI configuration files and scripts (PR prefix ci)
  • Documentation only changes (PR prefix docs)

How Has This Been Tested?

  • pre-commit tested in repo where we have custom chart and it lints and failes if linting is not passing, also README generated automatically
  • pre-commit tested in repo, if does not have helm chart, all passing, for example for external-secrets:
~/code/github.com/lablabs/modules/new/terraform-aws-eks-external-secrets 
.venv ❯ pre-commit run --all-files
trim trailing whitespace.................................................Passed
check for merge conflicts................................................Passed
detect aws credentials...................................................Passed
detect private key.......................................................Passed
fix end of files.........................................................Passed
Terraform validate.......................................................Passed
Terraform fmt............................................................Passed
Lock terraform provider versions.........................................Passed
Terraform validate with tflint...........................................Passed
Checkov..................................................................Passed
Terraform docs...........................................................Passed
Detect secrets...........................................................Passed
zizmor...................................................................Passed
Sync module variables....................................................Passed
Helm Lint................................................................Passed
Generate Helm Chart README with helm-docs................................Passed

@atyu1 atyu1 self-assigned this Jun 4, 2025
@atyu1 atyu1 requested a review from jaygridley June 4, 2025 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants