feat (rql): add ilike and notilike support by paanSinghCoder · Pull Request #83 · raystack/salt
📝 Walkthrough
Walkthrough
Extended string filter operator validation in the RQL parser to support case-insensitive variants ilike and notilike. The change expands the validStringOperations list while maintaining existing validation logic and control flow.
Changes
| Cohort / File(s) | Summary |
|---|---|
String Operator Validation rql/parser.go |
Added ilike and notilike case-insensitive operator variants to the validStringOperations validation set alongside existing like and notlike operators. |
Estimated code review effort
🎯 1 (Trivial) | ⏱️ ~3 minutes
Poem
🐰 A parser's delight, both subtle and bright,
Case-insensitive variants now in sight!
Like becomes ilike, a flexible friend,
Where queries bend, there's no end—
More operators to parse with glee! ✨
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
| Check name | Status | Explanation |
|---|---|---|
| Title check | ✅ Passed | The title clearly and concisely summarizes the main change: adding ilike and notilike operators to support case-insensitive string filtering in the RQL parser. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing Touches
📝 Generate docstrings
- Create stacked PR
- Commit on current branch
🧪 Generate unit tests (beta)
- Create PR with unit tests
- Commit unit tests in branch
feat/add-ilike-notilike-support
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands and usage tips.