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

Consistently use org.jooq Maven groupId across the manual #13556

Closed
aru-trackunit opened this issue May 13, 2022 · 3 comments
Closed

Consistently use org.jooq Maven groupId across the manual #13556

aru-trackunit opened this issue May 13, 2022 · 3 comments

Comments

@aru-trackunit
Copy link

Hi,

Dependency from the very bottom of the documentation

<groupId>org.jooq.trial</groupId>
<artifactId>jooq-meta-extensions</artifactId>
<version>3.12.4</version>

should be replaced with the one below

<groupId>org.jooq</groupId>
<artifactId>jooq-meta-extensions</artifactId>
<version>3.12.4</version>

See: https://www.jooq.org/doc/3.12/manual/code-generation/codegen-ddl/

@lukaseder
Copy link
Member

You have omitted the comments that are there in the documentation:

<dependency>
  <!-- Use org.jooq            for the Open Source Edition
           org.jooq.pro        for commercial editions, 
           org.jooq.pro-java-8 for commercial editions with Java 8 support,
           org.jooq.pro-java-6 for commercial editions with Java 6 support,
           org.jooq.trial      for the free trial edition 
                 
       Note: Only the Open Source Edition is hosted on Maven Central. 
             Import the others manually from your distribution -->
  <groupId>org.jooq.trial</groupId>
  <artifactId>jooq-meta-extensions</artifactId>
  <version>3.12.4</version>
</dependency>

So, this seems to sufficiently explain what needs to be done?

@lukaseder
Copy link
Member

Hmm, I've just noticed now that the default groupId is inconsistent across the manual. Some sections default to the org.jooq.trial groupId, others to the org.jooq one:

image

I guess we could consistently use the org.jooq one everywhere...

@lukaseder lukaseder changed the title Feedback about manual section "codegen-ddl" Consistently use org.jooq Maven groupId across the manual May 13, 2022
@lukaseder
Copy link
Member

Fixed. The changes will be deployed with the next manual build, soon.

Thanks again for your report.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants