-
Notifications
You must be signed in to change notification settings - Fork 6
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
Updated the ansible lint check workflow file #75
Conversation
Signed-off-by: sakethanne <[email protected]>
.github/workflows/ansible.yml
Outdated
# Install/Update Ansible and Ansible-lint | ||
- name: Install Ansible | ||
run: | | ||
python3 -m pip install ansible ansible-lint |
Check warning
Code scanning / Scorecard
Pinned-Dependencies Medium
Click Remediation section below to solve this issue
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.
why not
- name: run-ansible-lint
uses: ansible/[email protected]
first ?
see https://ansible.readthedocs.io/projects/lint/installing/#installing-ansible-lint-as-a-github-action
and warning
A new release of ansible-lint is available: 24.5.0 → 24.9.2
I tried that initially, but it is still unable to recognize the installed collections. with the new script we do not need to specify the version, as it automatically picks up the latest version available |
maybe we need to pass argument like
|
when I try to install the collections, it says they are already installed
|
how you install them ? manually ? or via argument to github action of linter ? |
I initially added this as a part of the code:
then removed it, since they are already installed |
but this https://ansible.readthedocs.io/projects/lint/installing/#installing-ansible-lint-as-a-github-action clearly states
I would try new |
Let me try the new option and get back with the results of it soon |
Signed-off-by: sakethanne <[email protected]>
Signed-off-by: sakethanne <[email protected]>
Signed-off-by: sakethanne <[email protected]>
Signed-off-by: sakethanne <[email protected]>
Signed-off-by: sakethanne <[email protected]>
Signed-off-by: sakethanne <[email protected]>
Signed-off-by: sakethanne <[email protected]>
Signed-off-by: sakethanne <[email protected]>
Signed-off-by: sakethanne <[email protected]>
Signed-off-by: sakethanne <[email protected]>
Signed-off-by: sakethanne <[email protected]>
fix(workflow): updated the ansible lint check workflow file