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
Added JMS source kamelet for Apache Artermis #363
Conversation
|
ef89c12
to
2dca891
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe change the name to follow the kamelets naming convention. The content is ok.
Cool, thanks for taking a look at it Andrea! I'll review the failing checks and put it according to the standard :) |
fb0dbb1
to
4afbaa2
Compare
1013e0f
to
eaf4337
Compare
This one is still broken. I am fixing it. |
eaf4337
to
a372eed
Compare
It should be fine now. It had an extra file. |
This and subsequent PRs will have conflicts on the nav file. I am watching for the approvals / comments and will be fixing the conflicts as they appear. |
b8e7b60
to
1a6aa98
Compare
Hey @oscerd, do we still have anything to change on this one? I think it's pretty much the same as the others ... I'm wondering if I missed anything. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm
Thanks! |
No description provided.