Skip to content

Fix some failures using wildfly-swarm profile#424

Merged
arjantijms merged 1 commit intojavaee-samples:masterfrom
juangon:SWARM-1675_2
Nov 20, 2017
Merged

Fix some failures using wildfly-swarm profile#424
arjantijms merged 1 commit intojavaee-samples:masterfrom
juangon:SWARM-1675_2

Conversation

@juangon
Copy link
Copy Markdown
Contributor

@juangon juangon commented Nov 17, 2017

-->by upgrading upgrading arquillian and wildfly swarm version. Additionally, cut the dependency from test-utils to parent to avoid unwanted added dependencies

Here we go @arjantijms !

Thanks a lot!

@arjantijms
Copy link
Copy Markdown
Contributor

Thanks! Looks great. I'll wait for travis to clear before merging, but looks okay!

@juangon
Copy link
Copy Markdown
Contributor Author

juangon commented Nov 17, 2017

Great. Your solution looks much better than mine and is the actual fix for the actual error. Thanks!

@juangon
Copy link
Copy Markdown
Contributor Author

juangon commented Nov 17, 2017

Oopps it fails as it tries to compile the project alone first. I think we unfortunately need to declare that variable in the test-utils project. Duplicate variable but will work...

@juangon
Copy link
Copy Markdown
Contributor Author

juangon commented Nov 17, 2017

Will update branch on Monday.

…arquillian and wildfly swarm version. Additionally, cut the dependency from test-utils to parent to avoid unwanted added dependencies
@juangon
Copy link
Copy Markdown
Contributor Author

juangon commented Nov 20, 2017

Can't believe, 5 hours to execute these tests and 3 still left :-0

@juangon
Copy link
Copy Markdown
Contributor Author

juangon commented Nov 20, 2017

@arjantijms ready to go!

@arjantijms
Copy link
Copy Markdown
Contributor

Can't believe, 5 hours to execute these tests and 3 still left :-0

It's indeed quite slow. I guess it's due to Travis being a free service and doesn't give you much cycles.

@arjantijms
Copy link
Copy Markdown
Contributor

@arjantijms ready to go!

\o/

@arjantijms arjantijms merged commit e8c9ed7 into javaee-samples:master Nov 20, 2017
@juangon
Copy link
Copy Markdown
Contributor Author

juangon commented Nov 20, 2017

Thanks very much @arjantijms !

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