Skip to content

chore(storage:inkless): add jooq classes#490

Merged
ivanyu merged 2 commits intomainfrom
jeqo/add-jooq-classes
Jan 22, 2026
Merged

chore(storage:inkless): add jooq classes#490
ivanyu merged 2 commits intomainfrom
jeqo/add-jooq-classes

Conversation

@jeqo
Copy link
Contributor

@jeqo jeqo commented Jan 21, 2026

To improve developer experience, jooq classes will be added to the source code instead of generating them on build. This will remove the strong dependency on Docker to build inkless.

jeqo added 2 commits January 21, 2026 20:24
To improve developer experience, jooq classes will be added to the
source code instead of generating them on build. This will remove the
strong dependency on Docker to build inkless.
@jeqo jeqo force-pushed the jeqo/add-jooq-classes branch 2 times, most recently from 88df437 to 746e284 Compare January 21, 2026 19:53
@jeqo jeqo marked this pull request as ready for review January 21, 2026 20:14
@jeqo jeqo requested a review from ivanyu January 21, 2026 20:14
@ivanyu ivanyu merged commit d3d9c1a into main Jan 22, 2026
10 checks passed
@ivanyu ivanyu deleted the jeqo/add-jooq-classes branch January 22, 2026 06:36
giuseppelillo pushed a commit that referenced this pull request Jan 27, 2026
* chore(inkless): Update make and ci to check built jooq classes

* chore(storage:inkless): add jooq classes

To improve developer experience, jooq classes will be added to the
source code instead of generating them on build. This will remove the
strong dependency on Docker to build inkless.
giuseppelillo pushed a commit that referenced this pull request Jan 27, 2026
* chore(inkless): Update make and ci to check built jooq classes

* chore(storage:inkless): add jooq classes

To improve developer experience, jooq classes will be added to the
source code instead of generating them on build. This will remove the
strong dependency on Docker to build inkless.
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.

2 participants