-
Notifications
You must be signed in to change notification settings - Fork 5
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
OPSEXP-2743 Run ansible-galaxy via pipenv when available #680
Conversation
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.
no versions.txt bump because you bump once the other PR is merged right?
Co-authored-by: Giovanni Toraldo <71768+gionn@users.noreply.github.com>
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.
The PR mentions testing Pipfile presence but the code test the cache dir. I think neither are really safe (though I admit would work in most case) as:
Pipfile location can be given from env var & cache directory can be cleared using pipenv --clear
and also set using env var.
Isn't it more simple to simply test the binary?
Checklist
Description