Skip to content

feat: Implement nil value handling in SQL query compilation#25

Merged
trheyi merged 1 commit into
YaoApp:mainfrom
trheyi:main
Apr 6, 2026
Merged

feat: Implement nil value handling in SQL query compilation#25
trheyi merged 1 commit into
YaoApp:mainfrom
trheyi:main

Conversation

@trheyi

@trheyi trheyi commented Apr 6, 2026

Copy link
Copy Markdown
Contributor
  • Added a new utility function filterNilBindings to filter out nil values from query bindings, ensuring that SQL queries are constructed correctly without including nil parameters.
  • Updated various query compilation methods across MySQL, PostgreSQL, and SQLite3 grammars to utilize the new nil handling logic, improving the robustness of query execution.
  • Enhanced unit tests to cover scenarios involving nil values in query parameters, ensuring comprehensive validation of the new functionality.

- Added a new utility function `filterNilBindings` to filter out nil values from query bindings, ensuring that SQL queries are constructed correctly without including nil parameters.
- Updated various query compilation methods across MySQL, PostgreSQL, and SQLite3 grammars to utilize the new nil handling logic, improving the robustness of query execution.
- Enhanced unit tests to cover scenarios involving nil values in query parameters, ensuring comprehensive validation of the new functionality.
@trheyi trheyi merged commit 11b10c0 into YaoApp:main Apr 6, 2026
2 of 3 checks passed
@github-actions

github-actions Bot commented Apr 6, 2026

Copy link
Copy Markdown

Thank you for the PR! The test workflow ( GO:1.24 DB:SQLite3 is running, the results of the run will be commented later.

@github-actions

github-actions Bot commented Apr 6, 2026

Copy link
Copy Markdown

Thank you for the PR! The test workflow ( GO:1.24 DB:Postgres14.0 is running, the results of the run will be commented later.

@github-actions

github-actions Bot commented Apr 6, 2026

Copy link
Copy Markdown

Thank you for the PR! The test workflow ( GO:1.24 DB:Postgres9.6 is running, the results of the run will be commented later.

@github-actions

github-actions Bot commented Apr 6, 2026

Copy link
Copy Markdown

Thank you for the PR! The test workflow ( GO:1.24 DB:MySQL8.0 is running, the results of the run will be commented later.

@github-actions

github-actions Bot commented Apr 6, 2026

Copy link
Copy Markdown

Thank you for the PR! The test workflow ( GO:1.24 DB:MySQL5.7 is running, the results of the run will be commented later.

@github-actions

github-actions Bot commented Apr 6, 2026

Copy link
Copy Markdown

✨DONE✨ GO:1.24 DB:SQLite3 passed.

@github-actions

github-actions Bot commented Apr 6, 2026

Copy link
Copy Markdown

✨DONE✨ GO:1.24 DB:Postgres14.0 passed.

@github-actions

github-actions Bot commented Apr 6, 2026

Copy link
Copy Markdown

✨DONE✨ GO:1.24 DB:Postgres9.6 passed.

@github-actions

github-actions Bot commented Apr 6, 2026

Copy link
Copy Markdown

✨DONE✨ GO:1.24 DB:MySQL5.7 passed.

@github-actions

github-actions Bot commented Apr 6, 2026

Copy link
Copy Markdown

✨DONE✨ GO:1.24 DB:MySQL8.0 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.

1 participant