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

A Guide to Outbound Open Source Software #20

Merged
merged 10 commits into from
Jul 4, 2024

Conversation

lichenmao
Copy link

Need review.

lichenmao and others added 8 commits March 22, 2024 01:02
Signed-off-by: maotiamo <xiangkemao@163.com>
Signed-off-by: maotiamo <71118570+lichenmao@users.noreply.github.com>
Signed-off-by: maotiamo <xiangkemao@163.com>
Signed-off-by: maotiamo <xiangkemao@163.com>
Signed-off-by: maotiamo <xiangkemao@163.com>
Signed-off-by: maotiamo <xiangkemao@163.com>
This reverts commit ab9c2c4.

Signed-off-by: maotiamo <xiangkemao@163.com>
Signed-off-by: maotiamo <xiangkemao@163.com>
Signed-off-by: maotiamo <xiangkemao@163.com>
@node
Copy link
Member

node commented Apr 22, 2024

不属于本PR 范围的 content/zh-CN/guides/ospo-simple-faq.md 文件被修改了,建议回退,否则提交后会覆盖其他译者的翻译。

@node
Copy link
Member

node commented Apr 22, 2024

由于是基于已有的英文文件来翻译,因为无需保留中英对照,仅保留中文即可;markdown相关的语法标记无需翻译,否则会影响编译和排版布局。

Copy link
Member

@node node left a comment

Choose a reason for hiding this comment

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

删除英文,仅保留中文。

@terizhao
Copy link

@lichenmao Please address comments. Thanks.

Signed-off-by: maotiamo <xiangkemao@163.com>
@lichenmao
Copy link
Author

@lichenmao Please address comments. Thanks.

OK, I have removed the English.

Copy link
Member

@node node left a comment

Choose a reason for hiding this comment

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

LGTM

@node
Copy link
Member

node commented Jul 1, 2024

本PR包含了对于其他文件的更新,请确保不会与其他PR相互覆盖;否则,建议在本PR中放弃对于其他文件的更新提交。

@zRich zRich merged commit f70ccb5 into lfapac-open-source-evangelist:main Jul 4, 2024
1 of 2 checks passed
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