-
Notifications
You must be signed in to change notification settings - Fork 95
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
all filter strings to lower case #1444
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.
Could you add a changelog entry for that and edit the PR's checklist accordingly?
Codecov Report
@@ Coverage Diff @@
## master #1444 +/- ##
=======================================
Coverage 36.71% 36.71%
=======================================
Files 989 989
Lines 22185 22185
Branches 6087 6110 +23
=======================================
Hits 8145 8145
Misses 12715 12715
Partials 1325 1325
Continue to review full report at Codecov.
|
Could you do me a favor and add two additional tests?
|
Amended FilterTerm to change all filter keywords to lowercase.
Checklist: