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
WINDUPRULE-422 @ContextName and @ContextNames have been removed #395
Conversation
Can one of the admins verify this patch? |
test this please |
@mrizzi do you know when this could be merged ? Thanks ! |
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.
I've just proposed a change to the effort to be consistent with RHAMT's docs.
If the change is fine for you @johnpoth, please just accept it and then the rule will be ready to be merged since it looks very good to me.
...-reviewed/camel3/camel2/java-multiple-camelcontexts-per-application-not-supported.windup.xml
Outdated
Show resolved
Hide resolved
…pplication-not-supported.windup.xml Co-Authored-By: Marco Rizzi <mrizzi@users.noreply.github.com>
@mrizzi those changes are fine with me! Thanks for reviewing! I'll update my code with the link you sent me regarding story points |
retest |
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.
The rule looks good @johnpoth, thanks a lot 👍
Thanks @mrizzi ! |
https://issues.redhat.com/browse/WINDUPRULE-422
Thanks !
cc @mrizzi