-
Notifications
You must be signed in to change notification settings - Fork 29
Update parent to 44 #67
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
base: master
Are you sure you want to change the base?
Conversation
Update of JUnit also, otherwise JUnit was not able to find tests (anymore)
src/site/site.xml
Outdated
<skin> | ||
<groupId>org.apache.maven.skins</groupId> | ||
<artifactId>maven-fluido-skin</artifactId> | ||
<version>2.1.0</version> | ||
</skin> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be taken from parent ....
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
version removed so taken from parent
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can remove whole tag for skin 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can remove whole tag for skin 😄
When I created the PR yesterday the built failed due the missing skin, that's why I added it:
https://github.com/apache/maven-source-plugin/actions/runs/14449578502/job/40518647343
Will check later again, when back home.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Local build with site works, let's see what CI says
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and there it fails:
Caused by: java.lang.NullPointerException: skin cannot be null
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried using the newest version of maven (as you said beta-4+ is needed) and of the api-dependencies (as there is no 4.0.0-rc3 for each of them).
<mavenVersion>4.0.0-rc-3</mavenVersion>
<version.maven.api-meta>4.0.0-rc-1</version.maven.api-meta>
<version.maven.api-impl>4.0.0-beta-5</version.maven.api-impl>
<version.maven.api-model>4.0.0-rc-3</version.maven.api-model>
<version.maven.api-di>4.0.0-rc-3</version.maven.api-di>
<version.maven.api-core>4.0.0-rc-3</version.maven.api-core>
But this fails with a lot of compilation errors thrown by the API plugins. Can you please give me a hint what to do if adding the skin into the site.xml is not wanted?
Update of JUnit also, otherwise JUnit was not able to find tests (anymore)