Skip to content

[MNG-8611] Build with apache-release profile#2146

Merged
gnodet merged 3 commits intoapache:masterfrom
gnodet:javadoc-release
Mar 5, 2025
Merged

[MNG-8611] Build with apache-release profile#2146
gnodet merged 3 commits intoapache:masterfrom
gnodet:javadoc-release

Conversation

@gnodet
Copy link
Contributor

@gnodet gnodet commented Mar 5, 2025

JIRA issue: MNG-8611

@gnodet gnodet changed the title Build with apache-release profile [MNG-8611] Build with apache-release profile Mar 5, 2025
@gnodet gnodet added this to the 4.0.0-rc-3 milestone Mar 5, 2025
@gnodet gnodet force-pushed the javadoc-release branch from c5d2460 to a508ed4 Compare March 5, 2025 08:44
Copy link
Member

@michael-o michael-o left a comment

Choose a reason for hiding this comment

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

Contains unrelated changes

@gnodet
Copy link
Contributor Author

gnodet commented Mar 5, 2025

Contains unrelated changes

@michael-o The first two commits are a test to make sure the problem is caught and that CI actually fails.

The problem is actually {@link } between reactor dependencies. The site build does not catch those problems, as the javadoc aggregates all modules together.

@gnodet gnodet requested a review from cstamas March 5, 2025 09:06
@gnodet gnodet requested a review from michael-o March 5, 2025 09:36
@gnodet gnodet merged commit ab102fb into apache:master Mar 5, 2025
13 checks passed
@michael-o
Copy link
Member

Contains unrelated changes

@michael-o The first two commits are a test to make sure the problem is caught and that CI actually fails.

The problem is actually {@link } between reactor dependencies. The site build does not catch those problems, as the javadoc aggregates all modules together.

OK, I see. Then it makes sense. It looked totally unrelated.

@jira-importer
Copy link

Resolve #9837

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants