Skip to content

github: add minimal reverse dependency test for ibcli#1711

Merged
thozza merged 1 commit into
osbuild:mainfrom
mvo5:reverse-deps-test-ibcli
Jul 30, 2025
Merged

github: add minimal reverse dependency test for ibcli#1711
thozza merged 1 commit into
osbuild:mainfrom
mvo5:reverse-deps-test-ibcli

Conversation

@mvo5

@mvo5 mvo5 commented Jul 30, 2025

Copy link
Copy Markdown
Contributor

This commit adds a minimal workflow to run the
image-builder-cli unit tests against the images library that is PRed.

This is inspired by the osbuild-composer integration but starts initially much simpler to avoid duplicating too much code, it would be nice to extract a common helper workflow.

This should give as a (non-required) test failure if we break ibcli accidentally.

[edit: this will fail until https://github.com//pull/1710 is merged]

@mvo5
mvo5 force-pushed the reverse-deps-test-ibcli branch 6 times, most recently from b8b7381 to 5d32c6d Compare July 30, 2025 11:50
@mvo5
mvo5 marked this pull request as ready for review July 30, 2025 12:02
@mvo5
mvo5 requested a review from a team as a code owner July 30, 2025 12:02
@thozza
thozza enabled auto-merge July 30, 2025 13:18
@supakeen

Copy link
Copy Markdown
Member

This will need a rebase after #1710 lands in main for the integration to pass :)

This commit adds a minimal workflow to run the
image-builder-cli unit tests against the images library
that is PRed.

This is inspired by the osbuild-composer integration
but starts initially much simpler to avoid duplicating
too much code, it would be nice to extract a common
helper workflow.
@bcl
bcl force-pushed the reverse-deps-test-ibcli branch from 5d32c6d to c3cc5c5 Compare July 30, 2025 22:19
@thozza
thozza added this pull request to the merge queue Jul 30, 2025
Merged via the queue into osbuild:main with commit 4670820 Jul 30, 2025
23 checks passed
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