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

CAMEL-21093 - Workaround to fix fhir auth x test due to commons-io #140

Conversation

apupier
Copy link
Contributor

@apupier apupier commented Aug 16, 2024

version clash

@apupier apupier mentioned this pull request Aug 16, 2024
@apupier
Copy link
Contributor Author

apupier commented Aug 19, 2024

@apupier
Copy link
Contributor Author

apupier commented Aug 19, 2024

the part which is not clear to me is why we need to add in the dependencyManagement of this example or the component although we are declaring the camel springboot bom

@davsclaus
Copy link
Contributor

Did you fix this in camel-fhir in the core project, or would this kind of PR still be needed here ?

@apupier
Copy link
Contributor Author

apupier commented Aug 26, 2024

seems the fix on camel core project is not working (or the snpashot artifact was still not pushed?) because the tests are still failing without this PR https://github.com/apache/camel-spring-boot-examples/actions/runs/10421144874?pr=137

@davsclaus davsclaus merged commit 48a31f6 into apache:main Sep 4, 2024
1 check passed
@davsclaus
Copy link
Contributor

lets merge to get the example working

mcarlett pushed a commit to mcarlett/camel-spring-boot-examples that referenced this pull request Sep 5, 2024
* fhir: move to test-infra

* fhir: update readme

Co-authored-by: John Poth <[email protected]>
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