Skip to content

Conversation

@hash-d
Copy link
Member

@hash-d hash-d commented Nov 24, 2025

This is one of two options for allowing to install the latest v1 CLI via install.sh.

The other one is #110.

On this option, a new 'v1-latest' version option is added.

Copy link
Member

@pwright pwright left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggestion from gemini

-H "X-GitHub-Api-Version: 2022-11-28" \
https://api.github.com/repos/skupperproject/skupper/releases \
| jq -r 'map(select( (.prerelease or .draft) | not )) | map(select(.name | startswith ("1."))) | map(select( .name | match("[0-9.]+") ) ) | sort_by (.name|split(".")|map(tonumber))[-1].name' \
>| "${release_version_file}"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

| operator: will crash the installer for non bash users, eg folks on Debian/Ubuntu systems.

Suggested change
>| "${release_version_file}"
> "${release_version_file}"


Versions:
latest The latest release
v1-latest The latest release in the 1.x line
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have the impression that if a user wants v1 they will specify a version (and that v1-latest is only useful for dev purposes).

@nluaces
Copy link
Member

nluaces commented Dec 3, 2025

I vote for #110 solution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants