Skip to content
This repository has been archived by the owner on Jan 13, 2023. It is now read-only.

Add basic test coverage - linting #16

Closed
geerlingguy opened this issue May 28, 2020 · 1 comment · Fixed by #17
Closed

Add basic test coverage - linting #16

geerlingguy opened this issue May 28, 2020 · 1 comment · Fixed by #17

Comments

@geerlingguy
Copy link
Owner

Just to have it.
geerlingguy added a commit that referenced this issue May 28, 2020
@geerlingguy
Fixes #16: Add ansible-lint and yamllint.
f2118af
@geerlingguy geerlingguy mentioned this issue May 28, 2020
Merged
@geerlingguy
Copy link
Owner Author

It seems Ansible Lint runs into issues with the short name resolution to FQCN of a collection defined inside a role's meta/main.yml: ansible/ansible-lint#644

So just doing yamllint for now.

geerlingguy added a commit that referenced this issue May 28, 2020
@geerlingguy
Merge pull request #17 from geerlingguy/16-lints
2afff86 
Fixes #16: Add ansible-lint and yamllint.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes #16: Add ansible-lint and yamllint. geerlingguy/turing-pi-cluster
1 participant
@geerlingguy