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.

Sign up for GitHub

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

Jump to bottom

Jetty 12 Documentation Html artifact not populated #9442

Closed
jmcc0nn3ll opened this issue Feb 27, 2023 · 2 comments
Closed

Jetty 12 Documentation Html artifact not populated #9442

jmcc0nn3ll opened this issue Feb 27, 2023 · 2 comments
Assignees
@olamy
Labels
Bug For general bugs on Jetty side Documentation

Comments

@jmcc0nn3ll
Copy link
Contributor

https://repo1.maven.org/maven2/org/eclipse/jetty/documentation/jetty-documentation/12.0.0.beta0/

The -html classifier artifact is not populated with HTML resulting in the Jetty website links being dead.
@jmcc0nn3ll jmcc0nn3ll added the Bug For general bugs on Jetty side label Feb 27, 2023
@jmcc0nn3ll jmcc0nn3ll added this to the 12.0.x milestone Feb 27, 2023
@sbordet sbordet removed this from the 12.0.x milestone Feb 28, 2023
@olamy
Copy link
Member

olamy commented Mar 8, 2023

ok I found the issue. arfff :)
the release script is skipping test using -DskipTests BUT the jetty-documentation module is skipping html generation if skipTests is true https://github.com/eclipse/jetty.project/blob/def35351f314cf8fe3f79ffd2c480ce3414d4a8b/documentation/jetty-documentation/pom.xml#L16

The solution is to add -Dasciidoctor.skip=false to the release script.

olamy added a commit that referenced this issue Mar 8, 2023
@olamy
Issue #9442 documentation html artifact does not contain html content
75dbba4 
Signed-off-by: Olivier Lamy <[email protected]>
olamy added a commit that referenced this issue Mar 9, 2023
@olamy
Issue #9442 documentation html artifact does not contain html content (…
3ac49c8 
…#9478)

Signed-off-by: Olivier Lamy <[email protected]>
@olamy
Copy link
Member

olamy commented Mar 9, 2023

fixed by 3ac49c8

@olamy olamy closed this as completed Mar 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@olamy olamy

Labels
Bug For general bugs on Jetty side Documentation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@olamy @joakime @jmcc0nn3ll @sbordet