Skip to content

Add the link of Chinese version of code reviewer's guide #8

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

Closed

Conversation

life1347
Copy link

@life1347 life1347 commented Sep 7, 2019

I tried to translate the code reviewer's guide into the Chinese version. Since the "Issues" tab is not visible, I created this PR to see if it's possible to add links for different language translation.
If it's not a good idea, let me know, I will close the PR.

And, thanks for creating this awesome repo.

Copy link

@limakzi limakzi left a comment

Choose a reason for hiding this comment

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

I completely disagree with this pull-request.

There are various reasons behind it.

  • There is no guarantee this website, as it looks like yours, will be permanent.
  • There is no guarantee, you will make this website up-to-date and keep it up-to-date.
  • I, having completely no clue on this beautiful language, have no skills to review what you wrote as most engineers. ☹️

What I would agree on:

  • Make a fork and maintain your own fork simultanously to the core.

@life1347
Copy link
Author

life1347 commented Sep 8, 2019

Thanks for your quick response. I agree with the points you list. I will close this one and maintain a fork as your suggestion.

@life1347 life1347 closed this Sep 8, 2019
@pauloedurezende
Copy link

pauloedurezende commented Sep 9, 2019

@limakzi So what would you recommend to others who want to translate to this repository?

  • Make a fork, perform the translation and then open a pull request with these changes for this project?

Or

  • Make a fork of this project, perform the translation, configure GitHub Pages and reference the translation in this project?

@mkanat mkanat reopened this Sep 9, 2019
@mkanat
Copy link
Contributor

mkanat commented Sep 9, 2019

While I appreciate the participation of @limakzi in this thread, he is not a reviewer or approver on this project. Let's leave this open for @adambender to review.

@limakzi
Copy link

limakzi commented Sep 9, 2019

@mkanat This is something what I wanted to write down after reading @pauloedurezende comment as it seems to be a misunderstanding. Hence I am not a @google employee or maintainer of this project, my review is kind of additional value to the project, as I really appreciate it open sourcing it. This is just my and only my own suggestion.

@adambender
Copy link
Collaborator

Hey all, new maintainer here. Im thrilled about having a Chinese version of the guide however, I need to find someone inside Google who can confirm the quality of the translation for me. Im going to let this PR stay open until I am able to do so.

@lightClouds917
Copy link

The Chinese version is a good idea. Many Chinese developers need this and can even offer multilingual versions.

Copy link

@lightClouds917 lightClouds917 left a comment

Choose a reason for hiding this comment

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

The Chinese version is a good idea!

@rootsongjc
Copy link

I have translated one https://jimmysong.io/eng-practices/

@lightClouds917
Copy link

I have translated one https://jimmysong.io/eng-practices/

Thank you for your contribution.

@life1347
Copy link
Author

life1347 commented Sep 19, 2019

@rootsongjc How about we merge two different Chinese translation versions into one?

I can be in charge of the traditional Chinese version (replace some phrases that more often used in Taiwan.) In this way, we can review the docs and polish the final result together to make sure that the output is close to the original one.

What do you think?

@rootsongjc
Copy link

@life1347 Based on which repo? Or create a new one?

@life1347
Copy link
Author

life1347 commented Sep 20, 2019

I just created an org https://github.com/eng-practices to host the repo. Since your repo structure is more close to the original one, how about you open a PR to this repo and then I base on the work to do the zh-tw translation?

We can create separate directories for different languages like

.
├── LICENSE
├── README.md
├── _config.yml
├── review
├── zh-cn
│   └── review
└── zh-tw
    └── review

I'm willing to add anyone who is interesting in helping to maintain the repo.
@adambender I'm thinking to add you to the org as well, let me know if it's a good idea. Thanks

@rootsongjc
Copy link

@life1347 OK, I'll take this.

@ninachen
Copy link
Collaborator

Hi there,
Apologies for the very long delay. I am the new owner of eng-practices and doing a review of open requests.

Thank you for making this huge contribution! We would be happy to incorporate it; I assume you are still interested in submitting this change.

If so, I will assign some Googler reviewers to the PR to do a final review: does that sound good to you?

@life1347
Copy link
Author

Hi @ninachen yes, that sounds good!

@rootsongjc and I have worked on the simplified Chinese version. I can close this one and open a new PR instead. Do you have any suggestions for how to organize the repo structure so that we can put the translated version into the right place?

Let me know if there is anything I can do to make the review process more smooth. And I would love to help to translate it into the traditional Chinese version as well.

@ninachen
Copy link
Collaborator

Sounds good! Your Googler reviewers will be @poponybing and @gavinbird.

Do you have any suggestions for how to organize the repo structure so that we can put the translated version into the right place?

Not at this time. Let's focus on the content first, and I and the other owners can figure out where the content should live separately -- no need to block the content review on figuring out the file organization. :)

@life1347
Copy link
Author

Close this one. See #20 instead.

@life1347 life1347 closed this Feb 24, 2020
@life1347 life1347 deleted the add-chinese-version-translation-link branch February 25, 2020 06:53
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.

8 participants