canonical / prometheus-k8s-operator

https://charmhub.io/prometheus-k8s
Apache License 2.0
21 stars 35 forks source link

Add yaml linting #481

Closed sed-i closed 1 year ago

sed-i commented 1 year ago

Issue

YAML files aren't linted.

Solution

Lint YAML files.

Context

Using yamllint may be obviated when:

Testing Instructions

Release Notes

Add yaml linting.

sed-i commented 1 year ago

Seems like this could be an argument in favor of a custom "generic o11y charm" pre-commit hook: instead of opening PRs on all repos, could have added yamllint just in one place.

sed-i commented 1 year ago

21 and #22 already capture all we need. Closing.