-
Notifications
You must be signed in to change notification settings - Fork 2.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove CiliumOperatorName constant #31597
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the submission! It looks like there are no longer any references to this constant. I think we can just remove it rather than correcting the typo.
1ac8543
to
dcde24d
Compare
I've amended my commit to do that instead! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
/test |
/test |
Do you need me to rebase maybe, or why are the tests failing? |
dcde24d
to
5862590
Compare
@thorn3r Rebased on main! |
/test |
1 similar comment
/test |
There are no longer any references to this constant so we should be able to remove it. Signed-off-by: Markus Nilsson <markus.nilsson@yubico.com>
5862590
to
8d6671b
Compare
/test |
Please ensure your pull request adheres to the following guidelines:
description and a
Fixes: #XXX
line if the commit addresses a particularGitHub issue.
Fixes: <commit-id>
tag, thenplease add the commit author[s] as reviewer[s] to this issue.