Skip to content
This repository has been archived by the owner on Nov 22, 2018. It is now read-only.

Create helm lint command #96

Closed
sgoings opened this issue Oct 27, 2015 · 5 comments
Closed

Create helm lint command #96

sgoings opened this issue Oct 27, 2015 · 5 comments
Assignees
Milestone

Comments

@sgoings
Copy link
Contributor

sgoings commented Oct 27, 2015

In order to help people create new charts and maintain a high standard of quality within the charts repo, I'd like to see a lint command within helm. The exact specifics of what "linting" mean are up for discussion but at this point @gabrtv and I have discussed:

  • is the Chartfile well formed?
  • are the manifests well-formed?
  • do the Docker images exist?
  • is the heritage of the Docker images trusted?
  • do the dependencies exist, and are the version constraints possible to match?

helm lint would then be run on charts that are submitted via PRs.

@technosophos
Copy link
Member

And "are the manifests well-formed"

@technosophos
Copy link
Member

"Do the dependencies exist, and are the version constraints possible to match?"

@michelleN
Copy link
Member

+1

@sgoings
Copy link
Contributor Author

sgoings commented Nov 3, 2015

Added both of @technosophos suggestions.

@michelleN
Copy link
Member

Some of the criteria in this issue have been moved to helm/helm#300

gabrtv pushed a commit to gabrtv/helm that referenced this issue Jun 30, 2016
introduce state of a resource to Resource in types.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants