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 yaml[octal-values] from skip_list #373

Merged
merged 1 commit into from
Jun 10, 2023
Merged

Fix yaml[octal-values] from skip_list #373

merged 1 commit into from
Jun 10, 2023

Conversation

ThomasSanson
Copy link
Contributor

No description provided.

✨ feat(ansible roles): wrap mode values in quotes to improve readability and consistency
The yaml[octal-values] rule was removed from the skip_list in the ansible-lint.yml file. The mode values in the ansible roles were wrapped in quotes to improve readability and consistency.

🐛 fix(roles): wrap mode values in quotes to avoid Ansible warnings
The mode values in the Ansible tasks were wrapped in quotes to avoid warnings from Ansible. This is a minor fix that improves the readability of the code and ensures that the tasks run without warnings.

🐛 fix(main.yml): wrap mode values in quotes to avoid Ansible deprecation warning
The mode values in the Ansible tasks were wrapped in quotes to avoid deprecation warnings. This is a minor fix that ensures the playbook runs without warnings.
@ThomasSanson
Copy link
Contributor Author

Hello @vitabaks,

Could you please restart the GitHub Docker action ?

@vitabaks
Copy link
Owner

Ок

@ThomasSanson
Copy link
Contributor Author

Great, it's done! Sometimes the issue comes from the timeout or the network during the Docker build, hence issue #374.

This pull request is ready.

@vitabaks
Copy link
Owner

Thank you for your contribution!

This work is very important for the further development of the project.

@vitabaks vitabaks merged commit 4f46f57 into vitabaks:master Jun 10, 2023
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