Skip to content

Add implicit query builder conversions from bool to BoolExpr#4547

Merged
joshua-spacetime merged 11 commits intomasterfrom
joshua/qb-from-bool
Mar 5, 2026
Merged

Add implicit query builder conversions from bool to BoolExpr#4547
joshua-spacetime merged 11 commits intomasterfrom
joshua/qb-from-bool

Conversation

@joshua-spacetime
Copy link
Collaborator

@joshua-spacetime joshua-spacetime commented Mar 4, 2026

Description of Changes

Adds implicit query builder conversions from bool to BoolExpr so that you can write:

ctx.from.user().r#where(|u| u.online)

instead of

ctx.from.user().r#where(|u| u.online.eq(true))

Also removes NullableCol and NullableIxCol types from C# query builder.

API and ABI breaking changes

None

Expected complexity level and risk

1

Testing

Unit and smoketests

@joshua-spacetime joshua-spacetime linked an issue Mar 4, 2026 that may be closed by this pull request
@joshua-spacetime joshua-spacetime requested a review from rekhoff March 4, 2026 07:24
Copy link
Contributor

@Shubham8287 Shubham8287 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed for Rust and Typescript, looks great.

@joshua-spacetime joshua-spacetime added this pull request to the merge queue Mar 5, 2026
Merged via the queue into master with commit c5743cf Mar 5, 2026
32 of 33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add query builder conversions from bool to BoolExpr

2 participants