Skip to content

Conversation

@gianpaj
Copy link
Contributor

@gianpaj gianpaj commented Oct 20, 2025

"Have fun coloring and learn about Podman!"

Before

image

After

image

@TomSweeneyRedHat
Copy link
Member

LGTM
thanks @gianpaj !

@TomSweeneyRedHat
Copy link
Member

@gianpaj our CI requires a signed commit to merge. Can you please git commit --amend -s, rebase and push again?

Copy link
Member

@lsm5 lsm5 left a comment

Choose a reason for hiding this comment

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

LGTM (pending DCO)

"Have fun coloring and learn about Podman!"

Signed-off-by: gianpaj <gianpa@gmail.com>
@gianpaj
Copy link
Contributor Author

gianpaj commented Nov 17, 2025

done. sorry for the delay

Copy link
Member

@lsm5 lsm5 left a comment

Choose a reason for hiding this comment

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

LGTM

@TomSweeneyRedHat
Copy link
Member

@gianpaj Thanks for the original find and fix, and then the DCO signing.
LGTM
Will merge once the ongoing tests are complete, as I expect them to.

@TomSweeneyRedHat TomSweeneyRedHat merged commit 8c89a8a into containers:main Nov 17, 2025
3 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