+
Skip to content

Conversation

stianst
Copy link
Contributor

@stianst stianst commented Oct 13, 2025

No description provided.

@stianst stianst requested a review from a team as a code owner October 13, 2025 14:41
Signed-off-by: Stian Thorgersen <stianst@gmail.com>
Copy link
Contributor

@ahus1 ahus1 left a comment

Choose a reason for hiding this comment

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

I think it needs some more tuning; see below.

- id: latest-release
run: |
branch="release/$(gh api repos/keycloak/keycloak/branches | jq -r '.[].name' | sort -r | awk -F'/' '/[0-9.]+$/ {print $NF; exit}')"
branch="$(gh api --paginate repos/keycloak/keycloak/branches -q '.[].name' | grep '^release/' | sort -r -V)"
Copy link
Contributor

@ahus1 ahus1 Oct 14, 2025

Choose a reason for hiding this comment

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

I ran this snippet:

branch="$(gh api --paginate repos/keycloak/keycloak/branches -q '.[].name' | grep '^release/' | sort -r -V)"

and it resulted in

$ echo $branch 
release/26.4 release/26.2 release/26.0

where it IMHO should return only a single item.

Maybe it is missing a head -1 to get just the first one.

Signed-off-by: Stian Thorgersen <stian@redhat.com>
@ahus1 ahus1 merged commit 33987e5 into main Oct 14, 2025
132 of 135 checks passed
@ahus1 ahus1 deleted the stianst-patch-1 branch October 14, 2025 08:32
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.

2 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载