Skip to content

Conversation

@ranjinimn
Copy link
Collaborator

@ranjinimn ranjinimn requested a review from e-minguez October 15, 2025 17:07
e-minguez
e-minguez previously approved these changes Oct 16, 2025
Copy link
Collaborator

@e-minguez e-minguez left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@hardys hardys left a comment

Choose a reason for hiding this comment

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

Small issue with the wording, otherwise lgtm - thanks!

Copy link
Contributor

@hardys hardys left a comment

Choose a reason for hiding this comment

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

lgtm, thanks!

@hardys hardys requested a review from e-minguez October 17, 2025 08:34
@hardys
Copy link
Contributor

hardys commented Oct 17, 2025

Please can we be careful to squash merge since the review feedback was added as a fixup commit and not locally squashed, thanks!

@e-minguez e-minguez merged commit 4ca2161 into suse-edge:main Oct 17, 2025
1 check passed
@ranjinimn ranjinimn deleted the ISSUE-896 branch October 17, 2025 11:32
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