Winduprule 475 spring integration quarkus

[jonathanvila]

https://issues.redhat.com/browse/WINDUPRULE-475

[PhilipCattanach]

@jonathanvila - I have reviewed the rules and executed the tests. The rules all fired. They are nicely worded and the issue report links work and are appropriate. Regarding rule # 00000 I think the effort needs to be changed from 10 in keeping with the RHAMT SP documentation. https://access.redhat.com/documentation/en-us/red_hat_application_migration_toolkit/4.3/html/cli_guide/reference_material#how_story_points_are_estimated_in_rules. Thanks

[jonathanvila]

Regarding rule # 00000 I think the effort needs to be changed from 10 in keeping with the RHAMT SP documentation.
+1 , changed to 7

[jonathanvila]

retest

[PhilipCattanach]

The rule tested successfully. Only one minor change required. Can the message for springboot-integration-to-quarkus-00000 be changed from "need" to "needs"
Thank you.

[mrizzi]

Other than the comments, this PR is missing the technology usage rules that #429 has.

[jonathanvila]

Other than the comments, this PR is missing the technology usage rules that #429 has.

@mrizzi : Spring Integration was already tackled by an old rule : https://github.com/windup/windup-rulesets/blob/master/rules-reviewed/technology-usage/integration.windup.xml#L45

[mrizzi]

Discussed with @PhilipCattanach the titles of the hints (see screenshot)

It's better to have consistency so please change Integration flow to become IntegrationFlow

[jonathanvila]

Discussed with @PhilipCattanach the titles of the hints (see screenshot)

It's better to have consistency so please change Integration flow to become IntegrationFlow

In fact that's on purpose..... IntegrationFlow is a class, Integration flow is the concept
@PhilipCattanach @mrizzi are you still wanting me to change it ?

[mrizzi]

Thanks @jonathanvila 👍