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
Don't skip doclint #891
Don't skip doclint #891
Conversation
165ca57
to
e554b96
Compare
(cherry picked from commit b5c1c0e)
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.
Works, but would seem to be inefficient?
@@ -485,6 +486,8 @@ | |||
<artifactId>maven-source-plugin</artifactId> | |||
<executions> | |||
<execution> | |||
<phase>compile</phase> |
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.
Why compile, this will be run on everysplit ATH.
Similar comment to below and the javadoc
Is this needed after #893? |
Given, the new task runs separately to the maven workflow to verify results independently, possibly not? |
The build as-is fails for me locally if I invoke
mvn clean verify -DskipTests
, and should fail on ci.j too, because we're lacking #890.