-
Notifications
You must be signed in to change notification settings - Fork 639
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
[Enhancement] Remove unused RocketMQ dependencies in List rocketmq
#4826
Comments
hello,I am new in there,I will try it,please assign me,thanks~ |
@heliang666s Welcome! You can start your work after #4719 is merged. |
请问已经有人解决了吗 |
@heliang666s #4719 has been merged already. |
ok,如果后续有新的简单的希望有机会 |
@yangsl960702 You can check out the Good first issue list, there are a lot of issues for freshmen. |
好的好的,非常感谢你,感动 |
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. |
@heliang666 May I ask that do you have any recent updates you can share with the community? |
Search before asking
Enhancement Request
As is described in #4719 (comment), there is no reference to the
rocketmq-broker
package in the code ofeventmesh-connector-rocketmq
module.Describe the solution you'd like
rocketmq-broker
package, there are some packages that are not being used by the code.eventmesh-connector-rocketmq
module, there some modules in which the sameList rocketmq
dependency set is being used.Streamline dependencies not used by the code.
Are you willing to submit PR?
Code of Conduct
The text was updated successfully, but these errors were encountered: