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

Add confluent jsonserde support #37870

Merged

Conversation

carlesarnal
Copy link
Contributor

Follow-up PR to #37722. Adds the confluent json schema serde support. I had to add a substitution and restrict some functionality because they're using some banned dependencies. Should not be merged until the main PR is merged.

@quarkus-bot quarkus-bot bot added area/core area/dependencies Pull requests that update a dependency file area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/documentation area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure area/kafka area/reactive-messaging area/smallrye labels Dec 20, 2023
@carlesarnal carlesarnal force-pushed the add-confluent-jsonserde-support branch 2 times, most recently from f67fd6b to e2e5b41 Compare December 21, 2023 09:03
Copy link

🎊 PR Preview 6884f92 has been successfully built and deployed to https://quarkus-pr-main-37870-preview.surge.sh/version/main/guides/

@carlesarnal carlesarnal marked this pull request as ready for review December 21, 2023 09:52

This comment has been minimized.

@carlesarnal
Copy link
Contributor Author

@cescoffier this is algo green on my fork. Thanks for merging the other one.

@gastaldi gastaldi requested a review from cescoffier January 2, 2024 11:17
@carlesarnal carlesarnal force-pushed the add-confluent-jsonserde-support branch from 0534658 to f0dd80f Compare January 2, 2024 15:33
@carlesarnal carlesarnal force-pushed the add-confluent-jsonserde-support branch from f0dd80f to 2700391 Compare January 2, 2024 15:34

This comment has been minimized.

Copy link

quarkus-bot bot commented Jan 7, 2024

✔️ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

@cescoffier cescoffier merged commit d4959b5 into quarkusio:main Jan 7, 2024
50 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.7 - main milestone Jan 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/core area/dependencies Pull requests that update a dependency file area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/documentation area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure area/kafka area/reactive-messaging area/smallrye
Projects
Development

Successfully merging this pull request may close these issues.

2 participants