Skip to content
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

v0.28 #1278

Draft
wants to merge 18 commits into
base: master
Choose a base branch
from
Draft

v0.28 #1278

wants to merge 18 commits into from

Conversation

igalklebanov
Copy link
Member

No description provided.

Copy link

vercel bot commented Nov 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
kysely ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 9, 2025 10:28am

Copy link

pkg-pr-new bot commented Nov 24, 2024

Open in Stackblitzkysely_koa_example

npm i https://pkg.pr.new/kysely-org/kysely@1278

commit: a619971

@belgattitude
Copy link

Btw thank you so much for the work you’re doing. Started using kysely and was really impressed !

congrats 🙌

@koskimas
Copy link
Member

@igalklebanov Feel free to merge this when you feel it's ready. I simply don't have the time right now to review. I trust you. Let me know when you want me to publish to npm.

@igalklebanov
Copy link
Member Author

Hey @koskimas 👋

Just parked it here to run CI (tests and preview release).
I wanna see if possible to add another long awaited feature before finalizing the list - and maybe some community PRs.

@cseickel
Copy link

cseickel commented Jan 3, 2025

@igalklebanov do you have any estimate on when v0.28 will be released? I'm itching to try out the performance improvements.

@igalklebanov
Copy link
Member Author

igalklebanov commented Jan 3, 2025

@igalklebanov do you have any estimate on when v0.28 will be released? I'm itching to try out the performance improvements.

We have preview releases. Try today.

npm i https://pkg.pr.new/kysely-org/kysely@1278

@flodlc
Copy link

flodlc commented Jan 4, 2025

@igalklebanov, thanks for the preview release.
Just tried it, I got JavaScript heap out of memory with 4gb allowed after a time approximately equal or higher than successful typecheck on version 0.27.4.
What time improvements should we expect? Is it an issue you already encountered?

Edit: with 8gb it works but I don't see any typecheck perf improvements on my side. I have around 60 tables and large codebase.

@igalklebanov igalklebanov linked an issue Jan 4, 2025 that may be closed by this pull request
@igalklebanov
Copy link
Member Author

igalklebanov commented Jan 4, 2025

@flodlc your comment is not actionable. Please submit a new issue with some traces, numbers, minimal reproduction, tsconfigs, TypeScript version, etc.

koskimas and others added 15 commits January 9, 2025 12:27
…#1085)

* add reusable helpers recipe and implement missing expression features

* force node 22.4.1 in CI because of an npm bug
* feat: add postgres range types (#1086)

* feat: support refresh naterialized view

* fix tests by adding .materialized() to remove the matview

* fix failing test

* fix: References typo (#1092)

* chore: refresh-view-node.ts => refresh-materialized-view-node.ts

* chore: export node in index.ts

---------

Co-authored-by: Isak <16906089+kansson@users.noreply.github.com>
Co-authored-by: Jonathan Wu <jonathanwu70@gmail.com>
Co-authored-by: Igal Klebanov <igal@grainfinance.co>
* feat: empty where in plugin

* test: add new tests

* chore: remove unneccesary typeguards

* fix: change to binary operator node

* test: update tests to do both in and not in

* test: for having

* chore: rm test

* test: nullable tests

* chore: nit

* chore: condense suite

* chore: db config override

* chore: extra console log

* chore: empty arr plugin docs

* HandleEmptyInListsPlugin initial commit.

Co-authored-by: Austin Woon Quan <43132101+austinwoon@users.noreply.github.com>

---------

Co-authored-by: Austin Woon <austin@open.gov.sg>
Co-authored-by: igalklebanov <igalklebanov@gmail.com>
remove only.
Co-authored-by: vincentiusvin <54709710+vincentiusvin@users.noreply.github.com>
Co-authored-by: Igal Klebanov <igalklebanov@gmail.com>
Co-authored-by: Ivashkin Olexiy <32811685+ivashog@users.noreply.github.com>
Co-authored-by: Dev K0te <dev.k0te42@gmail.com>
Co-authored-by: igalklebanov <igalklebanov@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change Includes breaking changes major-release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[SQLite Dialect] Properly translate .ignore() "IN" on empty array causes invalid SQL
10 participants