search

chaisql / chai

Modern embedded SQL database
MIT License
1.56k stars 95 forks source link

Add LOWER builtin function #510

Closed agonist closed 1 year ago

agonist commented 1 year ago

Adds LOWER to builtins functions to format TEXT fields to lower-case.

For non TEXT fields it will return NULL.

Types that can be casted to TEXT will work if they're first explicitly casted.

codecov[bot] commented 1 year ago

Codecov Report

Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (f94c703) 79.09% compared to head (b00e4ea) 79.04%.

:exclamation: Current head b00e4ea differs from pull request most recent head 53799fe. Consider uploading reports for the commit 53799fe to get more accurate results

Files Patch % Lines
internal/expr/functions/builtins.go 57.14% 8 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #510 +/- ## ========================================== - Coverage 79.09% 79.04% -0.05% ========================================== Files 125 125 Lines 10795 10816 +21 ========================================== + Hits 8538 8550 +12 - Misses 1541 1549 +8 - Partials 716 717 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.