-
-
Notifications
You must be signed in to change notification settings - Fork 86
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
[actions] Add drop to jdk 8 and maven 3.2.5 to run integration tests after initial build completion #582
Conversation
hazendaz
commented
May 29, 2023
- Build will run with respective Maven in wrapper and Java designated version first.
- Using the already built binary of license plugin, build will download lowest known to work maven version 3.2.5 using mvn, not the wrapper so maven version may vary here. This is done this way because windows won't allow wrapper to download wrapper due to jar being open.
- Finally integration tests baked into build will run against java 8 and maven 3.2.5
…after initial build completion - Build will run with respective Maven in wrapper and Java designated version first. - Using the already built binary of license plugin, build will download lowest known to work maven version 3.2.5 using mvn not the wrapper so maven version may vary here. This is done this way because windows won't allow wrapper to download wrapper due to jar being open. - Finally integration tests baked into build will run against java 8 and maven 3.2.5
@mathieucarbou This is a rework of original work I had been suggesting so we can drop jdk 8 from the main build process. If you look at the action steps on the run, it should be more clear as to what is going on. Also the original demo I showed off was completely wrong as it was not actually running the integration tests. It is now. Due to just trying to better demonstrate this, I have not removed or updated checkstyle yet. This seemed enough for now to show this. I further dropped to maven 3.2.5 from original try of maven 3.3.9. There is at least one maven plugin (clean plugin) that requires maven 3.2.5 or better. Maven itself is pushing that 3.2.5 or better now in general and likely to go higher soon. I don't know if there are really any usage stats of which maven versions but realistically anything before 3.6.x probably isn't or should not generally be used. |
run: ./mvnw clean install site -V -B -D"maven.artifact.threads=64" -D"org.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn" | ||
- name: Set up JDK 8 Integration Run (Consumer Run) |
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.
My understanding is that the second task will build and not re-use the previously workspace from the first task right ?
So if we have this additionnel task that is building and running with mv n3.2.5 / java 8, why do we need to keep the jdl 8 in the matrix above ?
Removal of jdk 8 would come next. Just setting up changes here.
Sent from my Verizon, Samsung Galaxy smartphone
Get Outlook for Android<https://aka.ms/AAb9ysg>
________________________________
From: Mathieu Carbou ***@***.***>
Sent: Tuesday, May 30, 2023 3:11:50 AM
To: mathieucarbou/license-maven-plugin ***@***.***>
Cc: Jeremy Landis ***@***.***>; Assign ***@***.***>
Subject: Re: [mathieucarbou/license-maven-plugin] [actions] Add drop to jdk 8 and maven 3.2.5 to run integration tests after initial build completion (PR #582)
@mathieucarbou commented on this pull request.
________________________________
In .github/workflows/ci.yaml<#582 (comment)>:
run: ./mvnw clean install site -V -B -D"maven.artifact.threads=64" -D"org.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn"
+ - name: Set up JDK 8 Integration Run (Consumer Run)
My understanding is that the second task will build and not re-use the previously workspace from the first task right ?
So if we have this additionnel task that is building and running with mv n3.2.5 / java 8, why do we need to keep the jdl 8 in the matrix above ?
—
Reply to this email directly, view it on GitHub<#582 (review)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AAHODI5YASTOMUAPIFIAAKTXIWMTNANCNFSM6AAAAAAYTHFZFM>.
You are receiving this because you were assigned.Message ID: ***@***.***>
|