Skip to content

Conversation

@perdasilva
Copy link
Contributor

This PR just addresses a comment in a previous PR where a unit test description was found to be mismatched

Signed-off-by: perdasilva perdasilva@redhat.com

@@ -0,0 +1,27 @@
package util
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this used?

Copy link
Contributor

Choose a reason for hiding this comment

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

(meaning the whole file)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh crap - nah that snuck in

Copy link
Contributor Author

Choose a reason for hiding this comment

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

was also trying to think about logging then gave up - ty!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed! @tmshort sorry for that - and thank you for catching it!

Signed-off-by: perdasilva <perdasilva@redhat.com>
Copy link
Member

@awgreene awgreene left a comment

Choose a reason for hiding this comment

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

/approve

@perdasilva perdasilva merged commit edca8be into operator-framework:main Jan 27, 2023
@perdasilva perdasilva deleted the small_fix branch January 27, 2023 13:54
LalatenduMohanty pushed a commit to LalatenduMohanty/operator-controller that referenced this pull request Dec 19, 2024
Signed-off-by: Catherine Chan-Tse <cchantse@redhat.com>
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