-
Notifications
You must be signed in to change notification settings - Fork 23
Expose option to download latest development release in the download actions #676
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
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.
@jprinet that is a very good point. I see two options:
I am leaning towards # 2 until we have a use case for # 1. So, we would have a boolean parameter like What do you think? |
I agree with your proposal, this is what we want at the moment and it won't pollute all the composite actions with repeated tedious logic |
9fc3ee3
to
831909b
Compare
@jprinet I've updated the PR according to our discussion. |
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 👍
There is a README page for each build tool, but I think it's fair to not document this parameter
Going to merge this. |
This PR adds an additional optional parameter to download the latest development release of the scripts. If left unspecified, the latest released version will be downloaded.