CNF-22645: Use strings.EqualFold for case-insensitive comparisons - #1117
CNF-22645: Use strings.EqualFold for case-insensitive comparisons#1117sebrandon1 wants to merge 1 commit into
Conversation
|
Hi @sebrandon1. Thanks for your PR. I'm waiting for a ComplianceAsCode member to verify that this patch is reasonable to test. If it is, they should reply with Regular contributors should join the org to skip this step. Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
@sebrandon1: This pull request references CNF-22645 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.22.0" version, but no target version was set. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
🤖 To deploy this PR, run the following command: |
dd6a517 to
f5ca590
Compare
|
🤖 To deploy this PR, run the following command: |
f5ca590 to
68948c8
Compare
|
🤖 To deploy this PR, run the following command: |
68948c8 to
703100c
Compare
|
🤖 To deploy this PR, run the following command: |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rhmdnd, sebrandon1 The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
703100c to
14f6a67
Compare
|
New changes are detected. LGTM label has been removed. |
|
🤖 To deploy this PR, run the following command: |
|
/retest |
1 similar comment
|
/retest |
14f6a67 to
bfedf83
Compare
|
🤖 To deploy this PR, run the following command: |
bfedf83 to
7bc6448
Compare
|
🤖 To deploy this PR, run the following command: |
c4556ee to
102abb1
Compare
|
🤖 To deploy this PR, run the following command: |
102abb1 to
f64083a
Compare
|
🤖 To deploy this PR, run the following command: |
f64083a to
b94c9b7
Compare
|
🤖 To deploy this PR, run the following command: |
b94c9b7 to
07ebec4
Compare
07ebec4 to
2ef777c
Compare
|
@sebrandon1: This pull request references CNF-22645 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "5.0.0" version, but no target version was set. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
🤖 To deploy this PR, run the following command: |
2ef777c to
22e635e
Compare
|
Hi @rhmdnd! Thanks for approving this PR back in April. I wanted to check in on the merge timeline — is there anything blocking this from being merged, or any additional changes you'd like to see? Happy to rebase if needed. Thanks! |
22e635e to
e063aba
Compare
|
🤖 To deploy this PR, run the following command: |
e063aba to
30b50eb
Compare
|
🤖 To deploy this PR, run the following command: |
Replace strings.ToLower(a) == strings.ToLower(b) with strings.EqualFold(a, b) in GetScanTypeIfValid and GetScannerTypeIfValid for cleaner, more efficient comparisons.
30b50eb to
525bad4
Compare
|
🤖 To deploy this PR, run the following command: |
|
@sebrandon1: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Summary
strings.ToLower(a) == strings.ToLower(b)withstrings.EqualFold(a, b)inGetScanTypeIfValid()andGetScannerTypeIfValid()Related PRs
Jira
Test Plan