Skip to content

modify/correct _zh-cn/index.md #2394

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

Merged
merged 87 commits into from
Sep 13, 2022
Merged

modify/correct _zh-cn/index.md #2394

merged 87 commits into from
Sep 13, 2022

Conversation

benluo
Copy link
Contributor

@benluo benluo commented May 10, 2022

modify/correct some translation in _zh-cn/index.md according to latest English version.

@eed3si9n eed3si9n requested a review from a team May 11, 2022 16:31
according to review.
Add _zh-cn/scala3/reference/README.md
Add _zh-cn/overview/scala3-book/why-scala-3.md
@benluo benluo requested a review from jxnu-liguobin May 14, 2022 03:33
@jxnu-liguobin
Copy link
Member

@benluo First of all, thank you for your work, it would be better if you could create new pull request. Because big PR isn't easy to be reviewed.

@benluo
Copy link
Contributor Author

benluo commented May 17, 2022

I don't know how to split my pull request. Any way, I will wait to commit my git until you review and merge all translation to main branch.

Copy link
Member

@jxnu-liguobin jxnu-liguobin left a comment

Choose a reason for hiding this comment

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

It's almost ready, a few small comments left.

@jxnu-liguobin
Copy link
Member

Hi, @benluo I see the some comments is closed, please resolve it or give a reasonable explanation.

@liufengyun It's almost done, so please take a look.

@benluo
Copy link
Contributor Author

benluo commented May 18, 2022

@jxnu-liguobin I modified translation according to all your comments in my latest commit. So i closed your comments without feedback. Please check my latest commit (300d5cd.

@jxnu-liguobin
Copy link
Member

@jxnu-liguobin I modified translation according to all your comments in my latest commit. So i closed your comments without feedback. Please check my latest commit (300d5cd.

sry. It was my misreading.

Copy link
Member

@jxnu-liguobin jxnu-liguobin left a comment

Choose a reason for hiding this comment

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

The following references obtained from 《Scala编程第5版》 can be tried out(Others who are not sure also can just refer to it.)

Union and intersection types并集类型和交集类型
typeclass 类型族
context bound 上下文界定
universal equality 通用相等性
muliversal equality 跨界相等性

benluo and others added 6 commits August 28, 2022 11:09
Co-authored-by: 梦境迷离 <568845948@qq.com>
Co-authored-by: 梦境迷离 <568845948@qq.com>
Co-authored-by: 梦境迷离 <568845948@qq.com>
Co-authored-by: 梦境迷离 <568845948@qq.com>
Co-authored-by: 梦境迷离 <568845948@qq.com>
@benluo benluo closed this Aug 28, 2022
@benluo
Copy link
Contributor Author

benluo commented Aug 28, 2022

I don't have 《Scala编程第5版》. Do you want to use all terms translation from that book?

@benluo benluo reopened this Aug 28, 2022
@jxnu-liguobin
Copy link
Member

Hi @SethTisue, Since this PR is really too huge (I'm not looking at it very carefully), do we need to split it into multiple smaller ones? or merge it first and then wait for other volunteers to modify it subsequently.

@SethTisue
Copy link
Member

Our usual approach with translations is to merge something even if it isn't perfect, and figure it can always be improved later.

The CI failure needs attention, though?

@benluo benluo requested a review from jxnu-liguobin September 2, 2022 15:29
@benluo
Copy link
Contributor Author

benluo commented Sep 2, 2022

Our usual approach with translations is to merge something even if it isn't perfect, and figure it can always be improved later.

The CI failure needs attention, though?

corrected.

@SethTisue SethTisue self-assigned this Sep 2, 2022
@bishabosha
Copy link
Member

this also looks like #2510, which should come first?

@jxnu-liguobin
Copy link
Member

this also looks like #2510, which should come first?

Yes, I think this pr should be merged at first.

@SethTisue SethTisue merged commit f256b1e into scala:main Sep 13, 2022
@SethTisue
Copy link
Member

thank you @benluo and @jxnu-liguobin !

@SethTisue SethTisue mentioned this pull request Sep 13, 2022
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.

4 participants