Skip to content

added case insensitive string comparison#8

Merged
abnegate merged 3 commits into
mainfrom
fix-case-sensitivity
Jul 21, 2026
Merged

added case insensitive string comparison#8
abnegate merged 3 commits into
mainfrom
fix-case-sensitivity

Conversation

@ArnabChatterjee20k

Copy link
Copy Markdown
Contributor

No description provided.

@greptile-apps

greptile-apps Bot commented Jul 21, 2026

Copy link
Copy Markdown

Greptile Summary

This PR adds case-insensitive string matching to conditions. The main changes are:

  • Case-insensitive equality for string values.
  • Case-insensitive string and array contains checks.
  • Case-insensitive prefix and suffix checks.
  • Tests for folded matching and case-sensitive ordering.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
src/Condition.php Updates string equality, contains, prefix, and suffix matching to fold case while leaving ordering comparisons case-sensitive.
tests/ConditionTest.php Adds coverage for folded string matching, scalar array membership compatibility, strict non-string equality, and case-sensitive ordering.

Reviews (3): Last reviewed commit: "keeping the case sensitiveness for compa..." | Re-trigger Greptile

Comment thread src/Condition.php Outdated
Comment thread src/Condition.php
@abnegate
abnegate merged commit 5f675e4 into main Jul 21, 2026
5 checks passed
@abnegate
abnegate deleted the fix-case-sensitivity branch July 21, 2026 06:14
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.

2 participants