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

Fix #635 Dependency parity check #638

Merged
merged 1 commit into from Jan 23, 2020

Conversation

ppalaga
Copy link
Contributor

@ppalaga ppalaga commented Jan 20, 2020

No description provided.

@lburgazzoli
Copy link
Contributor

looks like there is a dependency issue

@ppalaga
Copy link
Contributor Author

ppalaga commented Jan 21, 2020

9ba95df rebased to trigger a new build to see whether that's just a transfer problem

@ppalaga
Copy link
Contributor Author

ppalaga commented Jan 21, 2020

ok to test

1 similar comment
@lburgazzoli
Copy link
Contributor

ok to test

@lburgazzoli
Copy link
Contributor

@ppalaga yet another conflict

@ppalaga
Copy link
Contributor Author

ppalaga commented Jan 21, 2020

f25de47 rebased

@lburgazzoli
Copy link
Contributor

@ppalaga still failing

@ppalaga
Copy link
Contributor Author

ppalaga commented Jan 22, 2020

@gsmet isn't this java.util.NoSuchElementException: Property user.country not found https://github.com/apache/camel-quarkus/pull/638/checks?check_run_id=401147339#step:4:2706 caused by or related to the mysterious Quarkus issue you refer to in de4fffc ?

It is passing for me locally.

@ppalaga
Copy link
Contributor Author

ppalaga commented Jan 23, 2020

227e642 rebased in hope for the strange locale error to disappear

@gsmet
Copy link
Contributor

gsmet commented Jan 23, 2020

@ppalaga it has been fixed here: quarkusio/quarkus#6681 .

@gsmet
Copy link
Contributor

gsmet commented Jan 23, 2020

So it should be fixed if you use 1.2.0.Final.

@lburgazzoli lburgazzoli merged commit 8fb71a4 into apache:master Jan 23, 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.

None yet

4 participants