Skip to content
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

[ISSUE #428]Fix Chinese annotation #431

Merged
merged 6 commits into from
Jul 16, 2021
Merged

[ISSUE #428]Fix Chinese annotation #431

merged 6 commits into from
Jul 16, 2021

Conversation

Roc-00
Copy link
Contributor

@Roc-00 Roc-00 commented Jul 10, 2021

Translate the Chinese to English on the common module.
Only the Chinese part of the notes has been translated, and the Chinese in the code has not changed.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Welcome to the Apache EventMesh (incubating) community!!
This is your first PR in our project. We're very excited to have you onboard contributing. Your contributions are greatly appreciated!

Please make sure that the changes are covered by tests.
We will be here shortly.
Let us know if you need any help!

Want to get closer to the community?

WeChat Group:
wechat_qr

Mailing Lists:

Name Description Subscribe Unsubscribe Archive
Users User support and questions mailing list Subscribe Unsubscribe Mail Archives
Development Development related discussions Subscribe Unsubscribe Mail Archives
Commits All commits to repositories Subscribe Unsubscribe Mail Archives

@qqeasonchen qqeasonchen changed the title [ISSUE #428]:The Chinese existing in the project was changed to English [ISSUE #428]Fix Chinese annotation Jul 14, 2021
@@ -21,6 +21,7 @@

/**
* 这个RETRY的意思是 客户端发现投递的消息它没有监听时, 告诉EventMesh 发往下一个, 重试几次以实现灰度 , 预留
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We may also need to remove this Chinese comment

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Got it.

@codecov-commenter
Copy link

Codecov Report

Merging #431 (fad0381) into develop (d8ac875) will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             develop    #431   +/-   ##
=========================================
  Coverage       8.49%   8.49%           
  Complexity       256     256           
=========================================
  Files            228     228           
  Lines          10783   10783           
  Branches         918     918           
=========================================
  Hits             916     916           
  Misses          9793    9793           
  Partials          74      74           
Impacted Files Coverage Δ
...g/apache/eventmesh/common/command/HttpCommand.java 67.56% <ø> (ø)
...otocol/http/body/client/HeartbeatResponseBody.java 0.00% <ø> (ø)
...ol/http/body/message/ReplyMessageResponseBody.java 0.00% <ø> (ø)
...ttp/body/message/SendMessageBatchResponseBody.java 0.00% <ø> (ø)
...p/body/message/SendMessageBatchV2ResponseBody.java 0.00% <ø> (ø)
...esh/common/protocol/http/common/ClientRetCode.java 0.00% <ø> (ø)
...tmesh/common/protocol/http/common/ProtocolKey.java 0.00% <ø> (ø)
...col/http/header/client/HeartbeatRequestHeader.java 66.66% <ø> (ø)
...protocol/http/header/client/RegResponseHeader.java 65.78% <ø> (ø)
...rotocol/http/header/client/UnRegRequestHeader.java 66.66% <ø> (ø)
... and 20 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d8ac875...fad0381. Read the comment docs.

@qqeasonchen qqeasonchen merged commit 30dd6f5 into apache:develop Jul 16, 2021
jjz921024 pushed a commit to jjz921024/incubator-eventmesh that referenced this pull request Jul 25, 2021
* The Chinese in the notes is translated into English.

* Translation improvement

* Translation improvement

* fix Chinese annotation on runtime module
xwm1992 pushed a commit to xwm1992/EventMesh that referenced this pull request Dec 27, 2021
* The Chinese in the notes is translated into English.

* Translation improvement

* Translation improvement

* fix Chinese annotation on runtime module
xwm1992 pushed a commit that referenced this pull request Aug 4, 2022
* The Chinese in the notes is translated into English.

* Translation improvement

* Translation improvement

* fix Chinese annotation on runtime module
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants