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

OF-1660: Generate javadoc artifacts as part of the verify phase #1245

Merged
merged 2 commits into from Jan 7, 2019

Conversation

GregDThomas
Copy link
Contributor

mvn verify will now generate javadoc and sources jar files too.

mvn package does not - as the generation can be slow which is a PITA during the dev/test cycle.

@GregDThomas
Copy link
Contributor Author

Note; still requires the artifact to be moved to the right place.

@guusdk
Copy link
Member

guusdk commented Jan 7, 2019

I'm not a big fan to attaching this to a Maven phase where this action is unexpected. Can't we instead use either a profile, or an explicit additional argument in the maven command?

@GregDThomas
Copy link
Contributor Author

I hear what you're saying; the default phase for packaging javadoc/source files is, well, "package" - I pushed it back to verify to cut down the dev/test cycle time.

Does the CI build things with a specific profile? Could it do so?

@guusdk
Copy link
Member

guusdk commented Jan 7, 2019

The CI does whatever is configured in the .travis.yml file. In the current state, the command that's executed is on line 13: script: mvn verify

It's deceptively easy to modify to whatever you want. :)

@GregDThomas
Copy link
Contributor Author

OK, re-worked this. There's now a new "ci" profile, which generates sources and javadoc JAR files. That's called when running on Travis. This still requires the artifact to be moved to the right place.

@guusdk
Copy link
Member

guusdk commented Jan 7, 2019

Thank you! Moving the artifact to the right place will first require us to move it into the Maven repository. If you have time left, maybe look at line 17?

@GregDThomas
Copy link
Contributor Author

Well, I changed that line too, I think it's right, but may need to be merged before anything happens?

@guusdk
Copy link
Member

guusdk commented Jan 7, 2019

It's probably a good thing that Travis does not deploy anything to our Maven repo based on an unmerged PR. :)

@guusdk guusdk merged commit 71f93ef into igniterealtime:master Jan 7, 2019
@GregDThomas GregDThomas deleted the OF-1660 branch February 10, 2019 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants