Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fix javadoc options / Use legacy mode to overcome <source> level issue #3231

Closed
wants to merge 9 commits into from

Conversation

hazendaz
Copy link
Contributor

  • options need to have additional arguments as an array rather than string
  • removed old source 8 on javadocs as no longer needed here.
  • Per javadoc 3.6.0 plugin, maven introduced <legacyMode> which overcomes bugs with javadocs that prevented source from being set higher as documented here. To replace the source attribute this is set to 'true'

@scordio scordio self-requested a review October 23, 2023 05:46
@scordio scordio self-assigned this Oct 23, 2023
@scordio scordio added this to the 3.26.0 milestone Dec 20, 2023
@scordio
Copy link
Member

scordio commented May 10, 2024

Hi @hazendaz, sorry for getting back to you so late!

Some of your changes have been applied as part of #3324. Still, I would apply the changes for additionalOptions you proposed (done in fe0045d).

However, I'm unsure if legacyMode is required. The build already excludes the module path and we haven't noticed any side effects since #3324.

scordio added a commit that referenced this pull request May 10, 2024
Cherry-picked from #3231.

Co-authored-by: Jeremy Landis <hazendaz@users.noreply.github.com>
@scordio scordio modified the milestones: 3.26.0, 3.27.0 May 10, 2024
Kruschenstein and others added 2 commits May 11, 2024 01:36
option needed to have additional arguments, removed old source 8 on javadocs as no longer needed here.
@hazendaz hazendaz force-pushed the main branch 2 times, most recently from 6177af8 to ec03b22 Compare May 14, 2024 23:34
@hazendaz hazendaz closed this May 14, 2024
@hazendaz hazendaz reopened this May 14, 2024
@hazendaz
Copy link
Contributor Author

closing this, will open new one separately against main.

@hazendaz hazendaz closed this May 14, 2024
@hazendaz
Copy link
Contributor Author

@scordio How exactly is modular path excluded? Javadoc have always ran modular checks. The legacyMode flag (poorly named) disables that.

@hazendaz
Copy link
Contributor Author

Rebased this and created new PR to main since this got switched to 3.x since last time it was worked on.

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.

None yet

9 participants