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

Change the dokka output format from the javadoc style to the native dokka style #27516

Merged
merged 1 commit into from
Aug 29, 2022
Merged

Change the dokka output format from the javadoc style to the native dokka style #27516

merged 1 commit into from
Aug 29, 2022

Conversation

evanchooly
Copy link
Member

The dokka format has come a long way since it first started. It's now quite clean and easy to read. This will be a more idiomatic experience for our kotlin users.

@quarkus-bot
Copy link

quarkus-bot bot commented Aug 25, 2022

Thanks for your pull request!

The title of your pull request does not follow our editorial rules. Could you have a look?

  • title should preferably start with an uppercase character (if it makes sense!)

This message is automatically generated by a bot.

@evanchooly evanchooly requested a review from geoand August 25, 2022 20:09
@quarkus-bot quarkus-bot bot added area/dependencies Pull requests that update a dependency file area/kotlin labels Aug 25, 2022
@evanchooly evanchooly changed the title change the dokka output format from the javadoc style to the native dokka style Change the dokka output format from the javadoc style to the native dokka style Aug 25, 2022
@evanchooly
Copy link
Member Author

image

@quarkus-bot

This comment has been minimized.

@quarkus-bot quarkus-bot bot added area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/panache area/rest labels Aug 26, 2022
Comment on lines +974 to +991
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-jar-plugin</artifactId>
<executions>
<execution>
<id>dokka-jar</id>
<phase>package</phase>
<goals>
<goal>jar</goal>
</goals>
<configuration>
<classifier>javadoc</classifier>
<classesDirectory>${project.build.directory}/dokka</classesDirectory>
<skipIfEmpty>true</skipIfEmpty>
</configuration>
</execution>
</executions>
</plugin>
Copy link
Contributor

Choose a reason for hiding this comment

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

How come is this needed?

Copy link
Member Author

Choose a reason for hiding this comment

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

The dokka goal only generates the docs. we need the extra step to create and attach the jar.

@quarkus-bot

This comment has been minimized.

@geoand
Copy link
Contributor

geoand commented Aug 26, 2022

The MicroProfile TCKs Tests failures are not related to this and have been fixed on main.

The Devtools Tests failures look related

@evanchooly
Copy link
Member Author

The Devtools Tests failures look related

ah! found a typo in the template text. rebased while I was at it to try to get the MP test fixes.

Comment on lines +974 to +991
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-jar-plugin</artifactId>
<executions>
<execution>
<id>dokka-jar</id>
<phase>package</phase>
<goals>
<goal>jar</goal>
</goals>
<configuration>
<classifier>javadoc</classifier>
<classesDirectory>${project.build.directory}/dokka</classesDirectory>
<skipIfEmpty>true</skipIfEmpty>
</configuration>
</execution>
</executions>
</plugin>
Copy link
Contributor

Choose a reason for hiding this comment

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

@famod, @gsmet do you know anything about the necessity of this?

Copy link
Member

Choose a reason for hiding this comment

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

AFAICS the dokka goal (unlike the javadocJar goal) is not creating a javadoc jar, which is compensated by this jar-plugin execution.

Looks like a limitation of the plugin: https://kotlin.github.io/dokka/1.7.10/user_guide/maven/usage/ (see goal descriptions and config options)

Copy link
Contributor

Choose a reason for hiding this comment

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

Gotcha, thanks

@geoand geoand merged commit 9ce8749 into quarkusio:main Aug 29, 2022
@quarkus-bot quarkus-bot bot added this to the 2.13 - main milestone Aug 29, 2022
@evanchooly evanchooly deleted the dokka-jar branch September 1, 2022 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dependencies Pull requests that update a dependency file area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/documentation area/kotlin area/panache area/rest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants