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

Move maven integration tests to dedicated job #11674

Merged
merged 1 commit into from
Aug 28, 2020

Conversation

geoand
Copy link
Contributor

@geoand geoand commented Aug 27, 2020

Similar to what we do with gradle integration tests

@boring-cyborg boring-cyborg bot added the area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure label Aug 27, 2020
@geoand geoand force-pushed the maven-tests-java11 branch from bc6fb90 to 089a971 Compare August 27, 2020 13:30
@geoand geoand changed the title [do not merge] Only run maven tests with the Java 11 build [do not merge] Move maven integration tests to dedicated job Aug 27, 2020
@geoand geoand force-pushed the maven-tests-java11 branch 3 times, most recently from faabb3b to 87b5ec4 Compare August 27, 2020 15:50
@gsmet
Copy link
Member

gsmet commented Aug 27, 2020

For the record, I think it's a good idea.

@geoand
Copy link
Contributor Author

geoand commented Aug 27, 2020

It didn't seem to have much of an effect though...

@geoand geoand force-pushed the maven-tests-java11 branch from 87b5ec4 to 5a9135b Compare August 27, 2020 21:58
@geoand geoand changed the title [do not merge] Move maven integration tests to dedicated job Move maven integration tests to dedicated job Aug 27, 2020
@geoand
Copy link
Contributor Author

geoand commented Aug 27, 2020

To be merged after rebased onto #11679

@geoand geoand force-pushed the maven-tests-java11 branch from 5a9135b to 66c61af Compare August 28, 2020 06:03
@geoand
Copy link
Contributor Author

geoand commented Aug 28, 2020

@gsmet can I get an approval for this now that #11679 has been merged?

@geoand geoand added this to the 1.8.0 - master milestone Aug 28, 2020
Similar to what we do with gradle integration tests
@geoand geoand force-pushed the maven-tests-java11 branch from 66c61af to de28af8 Compare August 28, 2020 12:38
@geoand geoand merged commit 0c85c00 into quarkusio:master Aug 28, 2020
@geoand geoand deleted the maven-tests-java11 branch August 28, 2020 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure area/maven
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants