-
Notifications
You must be signed in to change notification settings - Fork 113
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
fix: covert toString before toLowercase #2830
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #2830 +/- ##
========================================
Coverage 87.25% 87.25%
========================================
Files 791 791
Lines 29195 29197 +2
Branches 6832 6833 +1
========================================
+ Hits 25473 25475 +2
Misses 3377 3377
Partials 345 345 ☔ View full report in Codecov by Sentry. |
@aashishmalik can we add more description about what issue does this PR solve & also add test-cases for the same ? |
Important Auto Review SkippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the To trigger a single review, invoke the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on X ? TipsChat with CodeRabbit Bot (
|
Kudos, SonarCloud Quality Gate passed! |
Description of the change
Type of change
Related issues
Checklists
Development
Code review