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

Disambiguate StringTemplate #2214

Merged
merged 1 commit into from
Mar 25, 2024
Merged

Disambiguate StringTemplate #2214

merged 1 commit into from
Mar 25, 2024

Conversation

akurtakov
Copy link
Contributor

@akurtakov akurtakov commented Mar 25, 2024

What it does

Fixes maven javadoc error "both class
org.eclipse.jdt.internal.compiler.ast.StringTemplate in org.eclipse.jdt.internal.compiler.ast and interface java.lang.StringTemplate in java.lang match" as can be seen at https://ci.eclipse.org/platform/job/eclipse.platform.releng.aggregator/job/master/2149/console

How to test

mvn clean verify javadoc:javadoc using Java 21 as JVM to run Maven completes without error as pointed above.

Author checklist

@merks
Copy link
Contributor

merks commented Mar 25, 2024

The parser appears to have the same problem:

image

adding the import fixes the problem too:

image

@akurtakov
Copy link
Contributor Author

Thanks @merks ! This actually didn't break the build as maven-javadoc-plugin skips internal packages but it would definitely bite us whenever the jump to Java 21 is done so I have ammended this PR to include it.

Fixes maven javadoc error "both class
org.eclipse.jdt.internal.compiler.ast.StringTemplate in
org.eclipse.jdt.internal.compiler.ast and interface
java.lang.StringTemplate in java.lang match" as can be seen at
https://ci.eclipse.org/platform/job/eclipse.platform.releng.aggregator/job/master/2149/console
@akurtakov akurtakov merged commit 76082af into eclipse-jdt:master Mar 25, 2024
9 checks passed
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

2 participants