Add YAML linter to Charts Pipelines
Overview
Add linting for YAML to the Charts pipelines to catch problems such as trailing whitespace.
The following discussion from !2167 (merged) should be addressed:
-
@dustinmm80 started a discussion: (+1 comment) I verified this by running a YAML linter that checks for trailing whitespace against this branch.
docker run --rm -it -v $(pwd):/data cytopia/yamllint charts/
This pointed me to some leftover trailing whitespace in
charts/gitlab-shell/templates/configmap-sshd.yml
andcharts/gitlab-shell/templates/configmap.yml
. I removed these spaces and pushed to this MR with !2167 (9f3bd6f3).We may want to consider adding this YAML lint step to CI so whitespace doesn't creep up on us again. @rmarshall how do you feel about this? Can be a followup issue.