Skip to content

[release-1.11] Add conditional release-checking system test#2643

Merged
mtrmac merged 1 commit into
podman-container-tools:release-1.11from
cevich:release-1.11_add_release_test
Jul 4, 2025
Merged

[release-1.11] Add conditional release-checking system test#2643
mtrmac merged 1 commit into
podman-container-tools:release-1.11from
cevich:release-1.11_add_release_test

Conversation

@cevich

@cevich cevich commented Jul 2, 2025

Copy link
Copy Markdown
Contributor

Unfortunately on a number of occasions, Skopeo has been released officially with a -dev suffix in the version number. Assist in catching this mistake at release time by the addition of a simple conditional test. Note that it must be positively enabled by a magic env. var. before executing the system tests.

Original PR: #2631

(AI was used to help craft this PR)

Unfortunately on a number of occasions, Skopeo has been released
officially with a `-dev` suffix in the version number.  Assist in
catching this mistake at release time by the addition of a simple
conditional test.  Note that it must be positively enabled by a
magic env. var. before executing the system tests.

Original PR: podman-container-tools#2631

Signed-off-by: Chris Evich <cevich@redhat.com>
@cevich
cevich marked this pull request as ready for review July 3, 2025 15:54

@lsm5 lsm5 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@mtrmac PTAL

@mtrmac mtrmac left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you!

@mtrmac
mtrmac merged commit c8ef2dc into podman-container-tools:release-1.11 Jul 4, 2025
8 checks passed
@stale-locking-app stale-locking-app Bot locked as resolved and limited conversation to collaborators Oct 3, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants