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

translate Chinese annotations to English annotations #2754

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

koko2pp
Copy link

@koko2pp koko2pp commented Sep 15, 2023

What problem does this PR solve?

Issue Number: #2576

Problem Summary: translate Chinese annotations to English annotations

What is changed and how it works?

What's Changed: The Chinese comments in the code files have been converted into English comments.

How it Works: Write a GO script. For various programming languages, extract all comments from code files using different regular expressions, then determine whether the comments contain Chinese characters. If there are Chinese characters, use the Baidu Translate API to translate the Chinese comments into English comments, and then write them back to the file.

Side effects(Breaking backward compatibility? Performance regression?): None

Check List

  • [1] Relevant documentation/comments is changed or added
  • [1] I acknowledge that all my contributions will be made under the project's license

@koko2pp
Copy link
Author

koko2pp commented Sep 18, 2023

cicheck

@aspirer
Copy link
Contributor

aspirer commented Sep 19, 2023

good job, I'll review it, but it's too many files changed, so may be a long time will be needed.

@aspirer aspirer self-requested a review September 19, 2023 06:46
* op data: refers to the data serialized by the request through protobuf
* @param request: Chunk Request
* @param data: The data content contained in the request
* @param log: Provide parameters, store serialized data, and ensure the data by the user= Nullptr
Copy link
Contributor

Choose a reason for hiding this comment

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

ensure the data!=Nullptr by the user

@aspirer
Copy link
Contributor

aspirer commented Sep 20, 2023

My review content:

  1. Checked all file changes to see if they were all changes in comments content.
  2. Randomly checked whether the translation of some comments is accurate. There is a small problem, which can be modified or not.

TODO for Contributor @koko2pp

  1. Ensure that CI passes. It may involve some cpplint, pylint and other static code format check failures. You can modify it as needed, or if it cannot be modified or is not suitable for modification, you can add // NOLINT comment.
  2. If possible, it is best to open up the translation tool you wrote and give us a link to the code base. This is voluntary.

Thank you very much for your contribution. Great work~

@aspirer
Copy link
Contributor

aspirer commented Sep 20, 2023

Tips:
Please do not use force push with an all in one commit. You can add a new commit so we can get what you changed based on last commit.
Otherwise, the changes between the two pushes cannot be confirmed. There are too many change files involved and it is impossible to review them one by one.

@wuhongsong wuhongsong added the repairing bug repairing label Oct 10, 2023
@Cyber-SiKu
Copy link
Contributor

cicheck

1 similar comment
@Cyber-SiKu
Copy link
Contributor

cicheck

@Cyber-SiKu
Copy link
Contributor

cicheck

1 similar comment
@Cyber-SiKu
Copy link
Contributor

cicheck

@koko2pp
Copy link
Author

koko2pp commented Oct 13, 2023

cicheck

1 similar comment
@Cyber-SiKu
Copy link
Contributor

cicheck

@SeanHai
Copy link
Contributor

SeanHai commented Nov 7, 2023

Maybe you can retry ci after rebase.

koko2pp and others added 8 commits November 22, 2023 11:51
Signed-off-by: koko2pp <spfrom1999@163.com>
change docs website

Signed-off-by: Wangpan <aspirer2004@gmail.com>

chore: prepare for rebase

style: Apply Clang format to modified files ,and restore patch files, and rebase all conficts.

style: Apply Clang format to modified files ,and restore patch files, and rebase all conficts.

style: Apply Clang format to modified files and restore patch files.

style: Apply Clang format to modified files ,and restore patch files, and rebase all conficts.

doc: add changelog for v2.7.

Signed-off-by: Wine93 <wine93.info@gmail.com>

Update CHANGELOG-2.7.md

Signed-off-by: Wangpan <aspirer2004@gmail.com>

update curve-arch.png

Signed-off-by: Wangpan <aspirer2004@gmail.com>

Code formatting changes

Signed-off-by: koko2pp <spfrom1999@163.com>
@koko2pp
Copy link
Author

koko2pp commented Nov 27, 2023

cicheck

@koko2pp
Copy link
Author

koko2pp commented Nov 27, 2023

cicheck

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
repairing bug repairing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants