Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: Configure javaTimeTypes #10

Merged
merged 1 commit into from
Jun 19, 2024
Merged

feat: Configure javaTimeTypes #10

merged 1 commit into from
Jun 19, 2024

Conversation

ursjoss
Copy link
Collaborator

@ursjoss ursjoss commented Jun 18, 2024

Adds a boolean field javaTimeTypes to JooqGeneratorConfig, defaulting to true.

This let's the user switch between the java.sql date/time types (if set to false) or the respective java.time types.

see https://www.jooq.org/doc/latest/manual/code-generation/codegen-advanced/codegen-config-generate/codegen-generate-java-time-types/

@ursjoss ursjoss requested a review from a team as a code owner June 18, 2024 21:12
Copy link
Contributor

@jcornaz jcornaz left a comment

Choose a reason for hiding this comment

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

LGTM

@jcornaz jcornaz merged commit 4a1e072 into main Jun 19, 2024
4 checks passed
@jcornaz jcornaz deleted the feat/javaTimeTypes branch June 19, 2024 08:26
This was referenced Jun 19, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants