Skip to content

v0.42.0 - cross joins, lateral joins, fixed lack of export on PgTextBuilderInitial type #4397

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Sukairo-02
Copy link
Collaborator

No description provided.

L-Mario564 and others added 3 commits April 10, 2025 06:14
* Add cross join support

* Add cross join tests

* Updated docs links (docs TBD)

* Merged cross join function types into join function types, fixed missing cases for `cross` join type, ordered `crossJoin` test queries

* `onIndex` support for `MySQL` `crossJoin`, fixed lack of `onIndex` param description in JSDoc, additional tests, disabled test case failing due to driver's issue

---------

Co-authored-by: Sukairo-02 <[email protected]>
Added new type MySqlJoinType which excludes the option for full joins and is used in the MySQL query builder createJoin function instead of the
regular JoinType.
Also removed two MySQL select type-tests which included the fullJoin function

Co-authored-by: Sukairo-02 <[email protected]>
@Sukairo-02 Sukairo-02 changed the title Nextver pr batch v0.42.0 - cross joins, lateral joins, fixed lack of export on PgTextBuilderInitial type Apr 10, 2025
* feat: added support for left and inner lateral join in postgres

Signed-off-by: Alex Vukadinov <[email protected]>

* chore: updated method descriptions on the new joins

Signed-off-by: Alex Vukadinov <[email protected]>

* GH-420 allowing sql inside lateral joins and revert package.json

* GH-420 reverted package.json empty line at the end

* Changed JSDoc [WIP], reversed check

---------

Signed-off-by: Alex Vukadinov <[email protected]>
Co-authored-by: Sukairo-02 <[email protected]>
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.

5 participants