[Enhancement] Remove unused RocketMQ dependencies in `List rocketmq`

apache / eventmesh

EventMesh is a new generation serverless event middleware for building distributed event-driven applications.

https://eventmesh.apache.org/

Apache License 2.0

1.6k stars 637 forks source link

[Enhancement] Remove unused RocketMQ dependencies in `List rocketmq` #4826

Open Pil0tXia opened 5 months ago

Pil0tXia commented 5 months ago

Search before asking

[X] I had searched in the issues and found no similar issues.

Enhancement Request

As is described in https://github.com/apache/eventmesh/pull/4719#discussion_r1560505036, there is no reference to the rocketmq-broker package in the code of eventmesh-connector-rocketmq module.

Describe the solution you'd like

In addition to the rocketmq-broker package, there are some packages that are not being used by the code.
In addition to the eventmesh-connector-rocketmq module, there some modules in which the same List rocketmq dependency set is being used.

Streamline dependencies not used by the code.

Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct *

heliang666s commented 5 months ago

hello,I am new in there,I will try it,please assign me,thanks~

Pil0tXia commented 5 months ago

@heliang666s Welcome! You can start your work after https://github.com/apache/eventmesh/pull/4719 is merged.

yangsl960702 commented 4 months ago

请问已经有人解决了吗

Pil0tXia commented 4 months ago

@heliang666s https://github.com/apache/eventmesh/pull/4719 has been merged already.

yangsl960702 commented 4 months ago

@heliang666s #4719 has been merged already.

ok，如果后续有新的简单的希望有机会

Pil0tXia commented 4 months ago

@yangsl960702 You can check out the Good first issue list, there are a lot of issues for freshmen.

yangsl960702 commented 4 months ago

@yangsl960702你可以看看Good first issues list，有很多适合新生的问题。

好的好的，非常感谢你，感动

github-actions[bot] commented 1 month ago

It has been 90 days since the last activity on this issue. Apache EventMesh values the voices of the community. Please don't hesitate to share your latest insights on this matter at any time, as the community is more than willing to engage in discussions regarding the development and optimization directions of this feature.

If you feel that your issue has been resolved, please feel free to close it. Should you have any additional information to share, you are welcome to reopen this issue.

Pil0tXia commented 1 month ago

@heliang666 May I ask that do you have any recent updates you can share with the community?