tigrisdata-archive / tigris-client-ts

TypeScript client for Tigris
https://www.tigrisdata.com/docs/sdkstools/typescript/
Apache License 2.0
15 stars 10 forks source link

fix: relax filter type checking #375

Closed ovaistariq closed 1 year ago

ovaistariq commented 1 year ago

What type of PR is this? (check all applicable)

Description

Currently, filters are strongly typed. This presents an issue for complex types like array, where we may want to do queries such as contains on array type fields.

Related Tickets & Documents

Added/updated tests?

Is this change backwards compatible?

Does it require updates to Tigris docs?

Checklist

[optional] Are there any post deployment tasks we need to perform?

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (47b7ce3) 90.25% compared to head (04d7e96) 90.26%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #375 +/- ## ======================================= Coverage 90.25% 90.26% ======================================= Files 29 29 Lines 5602 5606 +4 Branches 658 658 ======================================= + Hits 5056 5060 +4 Misses 545 545 Partials 1 1 ``` | [Impacted Files](https://app.codecov.io/gh/tigrisdata/tigris-client-ts/pull/375?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tigrisdata) | Coverage Δ | | |---|---|---| | [src/types.ts](https://app.codecov.io/gh/tigrisdata/tigris-client-ts/pull/375?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tigrisdata#diff-c3JjL3R5cGVzLnRz) | `91.41% <100.00%> (+0.04%)` | :arrow_up: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

github-actions[bot] commented 1 year ago

:tada: This PR is included in version 1.2.3 :tada:

The release is available on:

Your semantic-release bot :package::rocket: