Skip to content

fix: 错译和润色 #1196

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 5 commits into from
Jun 30, 2024
Merged

fix: 错译和润色 #1196

merged 5 commits into from
Jun 30, 2024

Conversation

omvjro
Copy link
Contributor

@omvjro omvjro commented Mar 1, 2024

目标章节
1-js/02-first-steps/05-types
1-js/09-classes/03-static-properties-methods
1-js/09-classes/05-extend-natives
1-js/09-classes/07-mixins
1-js/10-error-handling/1-try-catch
1-js/10-error-handling/2-custom-errors
1-js/11-async/01-callbacks
1-js/11-async/02-promise-basics
1-js/11-async/05-promise-api

当前上游最新 commit:无

本 PR 所做更改如下:

文件名 参考上游 commit 更改(理由)
article.md 修改部分错译,修改一些可能造成理解困难或不通顺的句子

@leviding
Copy link
Member

leviding commented Mar 3, 2024

非常感谢帮忙润色,最近有些忙,尽快抽时间 review 处理

@omvjro omvjro marked this pull request as draft March 6, 2024 12:43
@omvjro
Copy link
Contributor Author

omvjro commented Mar 6, 2024

我还没读完就发了这个pr,但最近发现可能还会有需要修改的地方,所以先转成draft了,不好意思

@leviding leviding marked this pull request as ready for review June 30, 2024 10:32
@leviding leviding merged commit 1c08dc3 into javascript-tutorial:master Jun 30, 2024
1 check passed
@leviding
Copy link
Member

@omvjro 谢谢,我做了几处修改后合并了,欢迎继续提 pull request

@leviding leviding added DONE and removed needs +1 labels Jun 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants