[Enhancement] Change to use switch instead of if-else.

[Alonexc]

[Enhancement] Change to use switch instead of if-else.

[kartiktayal]

I would like to pick this task ✋

[Alonexc]

Ohhh, that doesn't need to be changed to switch.

[kartiktayal]

And I think this should be simple enough so I would want to work on one more task if you would be kind enough to recommend one for me :)

[kartiktayal]

For below line,
https://github.com/apache/eventmesh/blob/master/eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/core/protocol/grpc/push/WebhookPushRequest.java#L190
I am getting the sonar validation error that there should be more conditions to use switch case and it is better to use if statement since we only have one condition.
Let me know if you still want me to change it to switch case.

[Alonexc]

@kartiktayal Hello and welcome to the eventmesh community, welcome contributions.

[Alonexc]

Hi, where is the modification? I did not understand, line 190?

[kartiktayal]

Hi, where is the modification? I did not understand, line 190?

For line 190, since there is only one condition, I was getting a sonar warning that it is better to use if else statement instead of switch case

[kartiktayal]

Ohhh, that doesn't need to be changed to switch.

I am open for another task as well if this is done.

[Alonexc]

You are very welcome to contribute to eventmesh!