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

Configure JitPack build with RAT and Chekstyle skipped #5

Closed
wants to merge 1 commit into from

Conversation

ledoyen
Copy link

@ledoyen ledoyen commented Mar 4, 2020

Hello,

I understand that this project is experimental and as such is not yet stable to get published to Central.

However I think that to have feedbacks it must be available in some way (other than building it locally, to avoid extra logic in CI descriptors).
To be honnest, I want to use it and see if it can improve how tests are done in the maven-dependency-plugin, (continuing maven-dependency-plugin#29).

So I tried JiPack which is great for that.

However the default command used by JitPack (mvn install -DskipTests) breaks.

So here is the configuration for JitPack that mimics the one used in GitHub actions.

@khmarbaise
Copy link
Owner

Hi,
to be honest it's not even stable enough to use it except as an experiments...but I would like to hear about the experience (there is a lot of stuff which does not yet work; Nor do I have good ideas at the moment)...

nevertheless I'm working on a branch currently (multimodule-build) which can be built with

mvn clean install -DskipTests -Dcheckstyle.skip=true 

Apart from that Jitpack might be an alternative for the experimental phase ....

@ledoyen
Copy link
Author

ledoyen commented Mar 4, 2020

This branch indeed works with JitPack:

<dependency>
    <groupId>com.github.khmarbaise.maven-it-extension</groupId>
    <artifactId>maven-jupiter-extension</artifactId>
    <version>827ff26</version>
</dependency>

Thanks for the quick answer.

I will let you decide whether you want to use JitPack or not and how.

@ledoyen ledoyen closed this Mar 4, 2020
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.

2 participants