Skip to content

Conversation

@NigelByrne1
Copy link
Contributor

What does this PR do?:

Which issue(s)/story(ies) does this PR fixes:

PR acceptance criteria:

  • Unit/Functional tests

  • Documentation

  • Client Impact

How to test changes / Special notes to the reviewer:

@NigelByrne1
Copy link
Contributor Author

/retest

Copy link
Member

@dirgim dirgim left a comment

Choose a reason for hiding this comment

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

LGTM

@openshift-ci openshift-ci bot removed the lgtm label Oct 30, 2025
@sonarqubecloud
Copy link

@dirgim
Copy link
Member

dirgim commented Oct 30, 2025

/retest

Copy link

@sonam1412 sonam1412 left a comment

Choose a reason for hiding this comment

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

lgtm on resolving Jiri's comment

Copy link
Member

@jsztuka jsztuka left a comment

Choose a reason for hiding this comment

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

lgtm

@openshift-ci
Copy link

openshift-ci bot commented Oct 30, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: dirgim, jsztuka, NigelByrne1, sonam1412

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@dirgim dirgim merged commit c7128eb into redhat-appstudio:main Oct 31, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants