Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use upstream Javadoc configuration #556

Merged
merged 2 commits into from
Jun 25, 2023
Merged

Use upstream Javadoc configuration #556

merged 2 commits into from
Jun 25, 2023

Conversation

basil
Copy link
Member

@basil basil commented Jun 25, 2023

This repository can be trivially made to conform to our upstream Javadoc configuration of <doclint>all,-missing</doclint>, removing the need to maintain a custom configuration and simplifying maintenance. Tested locally with mvn clean verify -Pjenkins-release.

@rantoniuk
Copy link
Contributor

Actually this generates 22 new JavaDoc warnings, but that's a good point, not much to get it aligned. I'll fix them and then merge this together.

@rantoniuk rantoniuk self-assigned this Jun 25, 2023
@basil
Copy link
Member Author

basil commented Jun 25, 2023

⇨ commit 1f18575

@rantoniuk rantoniuk merged commit 40bb324 into jenkinsci:master Jun 25, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants