Skip to content

Added remaining javadoc comments. - #1293

Merged
andrzejj0 merged 1 commit into
mojohaus:masterfrom
andrzejj0:more-javadoc
Oct 19, 2025
Merged

andrzejj0 merged 1 commit into
mojohaus:masterfrom
andrzejj0:more-javadoc

Conversation

@andrzejj0

@andrzejj0 andrzejj0 commented Oct 15, 2025 •

Copy link
Copy Markdown
Contributor

Also:

  • Supressing javadoc linter problems during report generation.
    - Additional javadoc linting step where these are suppressed (it's not possible to have two executions in reporting)

Currently I'm also enforcing Javadoc correctness, but I'm thinking whether it's not better to just have a lenient linter everywhere.

@slawekjaranowski please approve/advise.

@andrzejj0 andrzejj0 self-assigned this Oct 15, 2025
@andrzejj0 andrzejj0 added documentation Improvements or additions to documentation maintenance Cleanup, refactoring or other internal change labels Oct 15, 2025
@andrzejj0
andrzejj0 marked this pull request as draft October 15, 2025 06:27
@andrzejj0

Copy link
Copy Markdown
Contributor Author

Looks like that I can't make javadoc plugin actually see already compiled dependencies; even rearranging modules in parent won't help. I'll remove the additional linter step in favor of a slightly less lenient checks.

@andrzejj0
andrzejj0 marked this pull request as ready for review October 15, 2025 07:24
- Supressing javadoc linter problems during report generation.
 - Additional javadoc linting step where these are suppressed (it's not possible to have two executions in reporting)
@andrzejj0
andrzejj0 merged commit 0c5ae8f into mojohaus:master Oct 19, 2025
@andrzejj0
andrzejj0 deleted the more-javadoc branch October 19, 2025 10:59
@github-actions github-actions Bot added this to the 2.19.2 milestone Oct 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation maintenance Cleanup, refactoring or other internal change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants