Skip to content

[MPLUGIN-433] Allow to disable link validation#162

Merged
kwin merged 3 commits intomasterfrom
feature/improve-javadoc-link-validation
Oct 27, 2022
Merged

[MPLUGIN-433] Allow to disable link validation#162
kwin merged 3 commits intomasterfrom
feature/improve-javadoc-link-validation

Conversation

@kwin
Copy link
Copy Markdown
Member

@kwin kwin commented Oct 26, 2022

Also validate internal links to javadocs in mojo/parameter description and deprecated

@kwin kwin marked this pull request as draft October 26, 2022 18:26
@kwin kwin force-pushed the feature/improve-javadoc-link-validation branch from 5f1418f to 1af093b Compare October 27, 2022 09:23
@kwin kwin marked this pull request as ready for review October 27, 2022 09:50
@slawekjaranowski
Copy link
Copy Markdown
Member

Please also rebase with current codebase ... build should will be ok 😄

Also validate internal links to javadocs in mojo/parameter description
and deprecated info
@kwin kwin force-pushed the feature/improve-javadoc-link-validation branch from 0c5d079 to a0a8dad Compare October 27, 2022 10:29
@kwin
Copy link
Copy Markdown
Member Author

kwin commented Oct 27, 2022

Please also rebase with current codebase

Done.

@slawekjaranowski
Copy link
Copy Markdown
Member

What do you think about adding IT test for such options?

By the way we don't have any IT tests for javadoc links ... or I don't see.

@kwin
Copy link
Copy Markdown
Member Author

kwin commented Oct 27, 2022

By the way we don't have any IT tests for javadoc links ... or I don't see.

Not end2end but there is a lot of unit tests and https://github.com/apache/maven-plugin-tools/blob/master/maven-plugin-tools-api/src/test/java/org/apache/maven/tools/plugin/javadoc/JavadocSiteIT.java which checks that the links being created actually work against the Javadocs being published by Oracle for the JDK.

@kwin
Copy link
Copy Markdown
Member Author

kwin commented Oct 27, 2022

I now also added a proper IT in 6b564c8 with some basic checks on generated links.

@kwin kwin force-pushed the feature/improve-javadoc-link-validation branch from 6b564c8 to 4654149 Compare October 27, 2022 14:35
@kwin kwin force-pushed the feature/improve-javadoc-link-validation branch from 4654149 to 5868f26 Compare October 27, 2022 14:37
@kwin kwin merged commit 339fc29 into master Oct 27, 2022
@kwin kwin deleted the feature/improve-javadoc-link-validation branch October 27, 2022 17:56
@jira-importer
Copy link
Copy Markdown

Resolve #731

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.

3 participants