-
Notifications
You must be signed in to change notification settings - Fork 3
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
oblt-cli/setup: version
input precedence
#73
Conversation
e66c3c6
to
ebcfe0b
Compare
version
over version-file
version
input precedence
else | ||
echo "[ERROR] ${OBLT_CLI_VERSION_FILE} file not found." | ||
echo "::error title=elastic/oblt-actions/oblt-cli/setup::version-file not found: ${version_file}" |
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.
If version
is nil, what's the reason this condition fails rather than installing the latest version?
I see # Downloads the latest release if OBLT_CLI_VERSION is not set
is not anymore the case.
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.
I removed the comment. But I think the behavior already changed in #54
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.
- uses: elastic/oblt-actions/oblt-cli/setup@v1
with:
github-token: ${{ secrets.PAT }}
version: ""
Maybe version: ""
is not valid but version: latest
as you pointed out earlier. Then this comment/error can be set in that particular case. WDYT?
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.
I think this change is a breaking change
For the reason mentioned in #73 (comment) Also, the |
It's not a breaking change because #69 has not been released yet.
This change makes the action prefer the
version
input if bothversion
andversion-file
are defined.Additionally, it will add a warning annotation.
It will also add a notice annotation informing you about the downloaded version.