Skip to content

ci(pre-commit-ansible): limit ansible-lint to ansible/#6882

Merged
mitsudome-r merged 1 commit intoautowarefoundation:mainfrom
masahiro-kubota:main
Mar 9, 2026
Merged

ci(pre-commit-ansible): limit ansible-lint to ansible/#6882
mitsudome-r merged 1 commit intoautowarefoundation:mainfrom
masahiro-kubota:main

Conversation

@masahiro-kubota
Copy link
Contributor

@masahiro-kubota masahiro-kubota commented Mar 9, 2026

Description

pre-commit-ansible was checking YAML files outside the ansible/ directory.

This happened because the ansible-lint hook was not explicitly scoped to ansible/.

This PR limits the ansible-lint target to ansible/ in .pre-commit-config-ansible.yaml.

#6886 intentionally breaks the indentation in docker/docker-compose.yaml to show that pre-commit-ansible incorrectly checks YAML files outside ansible/, so I deleted them.

image

Since ansible-lint is now explicitly scoped to ansible/, the previous exclude_paths entries for directories outside ansible/ are no longer necessary.

How was this PR tested?

pre-commit-ansible ignores non-Ansible YAML file (docker/docker-compose.yaml) after this change

image

pre-commit-ansible still detects invalid YAML under ansible/

image

Signed-off-by: Masahiro Kubota <norikenpi@gmail.com>
@github-actions
Copy link

github-actions bot commented Mar 9, 2026

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

@mitsudome-r mitsudome-r merged commit ab8c29f into autowarefoundation:main Mar 9, 2026
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run:health-check Run health-check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants