Skip to content
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

DAT-18307 DevOps :: Extension release fails when 2 drafts have the same name #235

Merged
merged 2 commits into from
Aug 5, 2024

Conversation

jandroav
Copy link
Contributor

@jandroav jandroav commented Aug 2, 2024

https://datical.atlassian.net/browse/DAT-18307

🔧 (os-extension-automated-release.yml): improve retrieval of draft release ID by sorting releases by created_at date and selecting the last one to ensure the newest draft release ID is used

…lease ID by sorting releases by created_at date and selecting the last one to ensure the newest draft release ID is used
@sayaliM0412
Copy link
Contributor

can we test this?

@jandroav
Copy link
Contributor Author

jandroav commented Aug 2, 2024

can we test this?

yes, you can execute curl -s -H "Accept: application/vnd.github+json" -H "X-GitHub-Api-Version: 2022-11-28" -H "Authorization: Bearer ${{ secrets.BOT_TOKEN }}" "https://api.github.com/repos/liquibase/${{ matrix.repository }}/releases" | jq -r '[.[] | select(.draft == true)] | sort_by(.created_at) | last | .id' with one repo after creating 2 drafts

@jandroav jandroav merged commit f14a3ca into main Aug 5, 2024
@jandroav jandroav deleted the DAT-18307 branch August 5, 2024 12:38
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