Skip to content

Conversation

@fiam
Copy link
Collaborator

@fiam fiam commented Jul 19, 2025

  • Add a yamllint based Docker image for running the yaml linter
  • Add workflow to lint YAML files in pull requests
  • Create a minimal starting configuration for the yaml linter
  • Update all yaml files to pass the linter

- Add a yamllint based Docker image for running the yaml linter
- Add workflow to lint YAML files in pull requests
- Create a minimal starting configuration for the yaml linter
- Update all yaml files to pass the linter
@docker docker deleted a comment from jeampyz Jul 28, 2025
@docker docker deleted a comment from jeampyz Jul 28, 2025
@docker docker deleted a comment from jeampyz Jul 28, 2025
@docker docker deleted a comment from jeampyz Jul 28, 2025
@docker docker deleted a comment from andiamang73-stack Jul 28, 2025
@docker docker deleted a comment from andiamang73-stack Jul 28, 2025
@docker docker deleted a comment from andiamang73-stack Jul 28, 2025
@docker docker deleted a comment from andiamang73-stack Jul 28, 2025
@@ -0,0 +1,8 @@
FROM davidanson/markdownlint-cli2:v0.18.1 AS markdownlint

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: empty line

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, thanks! It should be fixed now, PTAL.

FROM davidanson/markdownlint-cli2:v0.18.1 AS markdownlint


FROM python:3.13-slim AS yamllint
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: python 3.14 ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3.14 is not yet out, check https://devguide.python.org/versions/

@fiam fiam merged commit 09bc107 into main Aug 6, 2025
19 checks passed
@fiam fiam deleted the alberto/yaml-lint branch August 6, 2025 07:19
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.

3 participants