Skip to content
New issue

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

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

Already on GitHub? # to your account

feat: add requirements_file input to GitHub action #4253

Merged
merged 2 commits into from
Jul 18, 2024

Conversation

dbrennand
Copy link
Contributor

@dbrennand dbrennand commented Jul 10, 2024

Summary

This PR implements a new optional input to the ansible-lint action named requirements_file.

This allows for users of the action to specify a path to a requirements file to install role and collection dependencies prior to running ansible-lint.

Usage example: https://github.com/dbrennand/home-ops/blob/dev/.github/workflows/ansible-lint.yml#L15
Sample run: https://github.com/dbrennand/home-ops/actions/runs/9879791162/job/27286802492

Fixes: #3636
Fixes: #3942

@dbrennand dbrennand requested a review from a team as a code owner July 10, 2024 19:04
@dbrennand dbrennand requested review from Qalthos and priyamsahoo and removed request for a team July 10, 2024 19:04
@jakub-laskowski-wttech
Copy link

jakub-laskowski-wttech commented Jul 15, 2024

Hello,
I have also tested this PR and is working on my repo. Thanks for fixing it.

@audgirka audgirka enabled auto-merge (squash) July 18, 2024 14:26
@audgirka audgirka disabled auto-merge July 18, 2024 14:38
@audgirka audgirka merged commit 26d83ea into ansible:main Jul 18, 2024
25 of 27 checks passed
@dbrennand dbrennand deleted the 3636 branch July 18, 2024 15:05
dbrennand added a commit to dbrennand/home-ops that referenced this pull request Jul 24, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
Archived in project
3 participants