camel-quarkus icon indicating copy to clipboard operation
camel-quarkus copied to clipboard

Published 20 hours ago • verified publisher icon apache

[Draft] Add cloudEvents and knative extensions

Open zbendhiba opened this issue 2 years ago • 1 comments

Fixes #3459

zbendhiba avatar Apr 07 '22 09:04 zbendhiba

@lburgazzoli could you check this PR? Is there a way to have more integration tests. I wonder if there's something we could use to simulate Knative, or do some mock.

zbendhiba avatar Apr 07 '22 09:04 zbendhiba

@jamesnetherton @lburgazzoli Is deleting the reflective classes mandatory for this PR?

zbendhiba avatar Oct 03 '22 13:10 zbendhiba

Let me know if something needed to explain or improve

zbendhiba avatar Oct 03 '22 13:10 zbendhiba

@jamesnetherton @lburgazzoli Is deleting the reflective classes mandatory for this PR?

I think it's ok to have the reflective config initially. But we should try to remove it in a future release if possible.

jamesnetherton avatar Oct 03 '22 13:10 jamesnetherton

Closed by mistake sorry

zbendhiba avatar Oct 03 '22 13:10 zbendhiba

@jamesnetherton @lburgazzoli Is deleting the reflective classes mandatory for this PR?

I think it's ok to have the reflective config initially. But we should try to remove it in a future release if possible.

Okay I'll open an issue and add it to my tasks if this is approved for next release :)

zbendhiba avatar Oct 03 '22 13:10 zbendhiba