-
Notifications
You must be signed in to change notification settings - Fork 409
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
ANSIENG-3627 Fix lint & doc tests #1592
ANSIENG-3627 Fix lint & doc tests #1592
Conversation
…with ignoring few lint checks
…mmand-syntax to no-free-form
…les role to fix lint errors
|
1 similar comment
|
…meta/main.yml for all roles
Please also add the meta file for variables role, so we don't face that warning |
Yes that is already present |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, Thanks!
Description
Few changes were needed to pass ansible-sanity and galaxy-importer tests which is necessary for releases on automation hub. To remove errors and warnings from those tests few linting changes are done here. Also molecule and test_roles have been added to ignore list for lint checks
Fixes # (ANSIENG-3627)
Type of change
How Has This Been Tested?
semaphore run
Checklist: