Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix linting #9

Merged
merged 11 commits into from
Nov 30, 2023
Merged

Fix linting #9

merged 11 commits into from
Nov 30, 2023

Conversation

p-j-smith
Copy link
Contributor

@p-j-smith p-j-smith commented Nov 30, 2023

Part of #8

  • move the verify-firewalld.yml task file into a separate folder, otherwise ansible-lint treats it like a playbook and errors
  • remove molecule folder from docker role - it had several linting issues, but the molecule setup is now in the tests/ folder at the collection level anyway (the docker molecule role setup was left in by mistake in the initial setup of this repo)
  • fix forbidden implicit octal values errors
  • fix missing starting space in comment errors
  • rename EXTERNAL_STORAGE_DRIVE to external_storage_drive
  • add var-naming[no-role-prefix] to warn list - we could fix these issues instead, but perhaps in a separate pr as it will require quite a lot of changes?
  • fix spelling mistakes

@p-j-smith p-j-smith requested a review from a team November 30, 2023 11:47
Copy link
Contributor

@drmatthews drmatthews left a comment

Choose a reason for hiding this comment

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

Looks good - let's fix variable naming in a separate PR.

@p-j-smith p-j-smith merged commit 4d95f95 into main Nov 30, 2023
2 checks passed
@p-j-smith p-j-smith deleted the maint/fix-linting branch November 30, 2023 11:55
p-j-smith added a commit to UCL-MIRSG/ansible-collection-xnat that referenced this pull request Dec 7, 2023
- to make Ansible-lint happy, `EXTERNAL_STORAGE_DRIVE` has been renamed
to `external_storage_drive`. This change was also made in the
[infrastructure collection when fixing some linting
issues](UCL-MIRSG/ansible-collection-infra#9).

- define `external_storage_drive` as an inventory variable rather than a
playbook one, otherwise we can't override it at the inventory level.
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.

2 participants