Skip to content

[MSITE-871] Upgrade Maven Javadoc Plugin in integration tests#39

Merged
elharo merged 2 commits intoapache:masterfrom
infosupport:MSITE-871-upgrade-m-j-p-in-integration-tests
Mar 1, 2021
Merged

[MSITE-871] Upgrade Maven Javadoc Plugin in integration tests#39
elharo merged 2 commits intoapache:masterfrom
infosupport:MSITE-871-upgrade-m-j-p-in-integration-tests

Conversation

@mthmulders
Copy link
Copy Markdown
Contributor

These test failed on macOS with Java above 8; now they pass with
Java 8, 11 and 15 on macOS.

These test failed on macOS with Java above 8; now they pass with
Java 8, 11 and 15 on macOS.
@bertysentry
Copy link
Copy Markdown
Contributor

This is cool, thank you!
We're making good progress as only 2 integration tests are now failing:

Error:  The following builds failed:
Error:  *  MSITE-512/pom.xml
Error:  *  MSITE-484/pom.xml

@elharo
Copy link
Copy Markdown
Contributor

elharo commented Mar 1, 2021

Running through Jenkins:

https://ci-builds.apache.org/job/Maven/job/maven-box/job/maven-site-plugin/job/msite-871/

Given that this fixes some tests, and does not break anything new I'm comfortable merging it. We can fix the remaining two in the next PR.

These test failed on macOS with Java above 8; now they pass with
Java 8, 11 and 15 on macOS.
@mthmulders
Copy link
Copy Markdown
Contributor Author

The remaining tests are also fixed by upgrading the Maven Javadoc Plugin. Just pushed.

@elharo
Copy link
Copy Markdown
Contributor

elharo commented Mar 1, 2021

Looks good. CI is slow today, but I'll merge as soon as they all pass. Thanks!

@bertysentry
Copy link
Copy Markdown
Contributor

Awesome news! :-)

@elharo elharo merged commit c03f624 into apache:master Mar 1, 2021
@jira-importer
Copy link
Copy Markdown

Resolve #1005

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants