Skip to content

fix: verify version download - #12

Merged
catsby merged 6 commits into
mainfrom
version-check-dl
Dec 17, 2024
Merged

fix: verify version download#12
catsby merged 6 commits into
mainfrom
version-check-dl

Conversation

@catsby

@catsby catsby commented Dec 17, 2024

Copy link
Copy Markdown
Contributor

fix for #11 by verifying the success of downloading the release, and error if not.

- match renovate syntax to uds-core
- update to newest version in github action

Signed-off-by: catsby <clint@defenseunicorns.com>
- correctly check for pre-step failure

Signed-off-by: catsby <clint@defenseunicorns.com>
Comment thread .github/workflows/setup-uds.yaml Outdated
Comment thread action.yaml Outdated
catsby added 4 commits December 17, 2024 14:27
Signed-off-by: catsby <clint@defenseunicorns.com>
Signed-off-by: catsby <clint@defenseunicorns.com>
Signed-off-by: catsby <clint@defenseunicorns.com>
Signed-off-by: catsby <clint@defenseunicorns.com>

@decleaver decleaver left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

when @mjnagel says its approved, its approved

@catsby
catsby merged commit ab842ab into main Dec 17, 2024
@catsby
catsby deleted the version-check-dl branch December 17, 2024 21:43
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