Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

ArrayBuffer, binary arrays#529

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
leviding merged 15 commits intojavascript-tutorial:masterfrombemself:master
Dec 5, 2019

Conversation

bemself
Copy link
Collaborator

@bemselfbemself commentedNov 20, 2019•
edited
Loading

目标章节：4-binary/01-arraybuffer-binary-arrays

当前上游最新 commit：54984506

所做更改如下：

文件名参考上游 commit更改（理由）
article.mdc5ce557新增翻译文件
o1-concat/task.md6fb4aab新增翻译文件
_js.view/solution.js6fb4aab新增翻译文件
_js.view/source.js6fb4aab新增翻译文件

leviding reacted with thumbs up emoji
@iliakaniliakan mentioned this pull requestNov 20, 2019
@bemselfbemself changed the titleArrayBuffer, binary arrays[WIP] ArrayBuffer, binary arraysNov 29, 2019
@bemselfbemself changed the title[WIP] ArrayBuffer, binary arraysArrayBuffer, binary arraysNov 29, 2019
@KerbalHao
Copy link
Collaborator

还请麻烦您修改一下 PR 内容，写上修改的文件和参考的上游 commit 等

@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@bemself
Copy link
CollaboratorAuthor

还请麻烦您修改一下 PR 内容，写上修改的文件和参考的上游 commit 等

已修改，谢谢~

/done

Copy link
Collaborator

@KerbalHaoKerbalHao left a comment

Choose a reason for hiding this comment

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

感谢翻译，校对完毕

@bemself
Copy link
CollaboratorAuthor

感谢校对，已修改~

/done

Copy link
Member

@levidingleviding left a comment

Choose a reason for hiding this comment

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

存在几点问题，麻烦译者@bemself 注意一下，并进行修改：

  1. 格式细节问题需要仔细
  2. 翻译成中文的，那就不要用英文的引号来扩中文汉字，符号也要相应改为中文全角
  3. 空格等细节格式问题
  4. 168 行的2<sup>8</sup> 是 28，也就是 2 的八次方，不是取余吧？

@KerbalHao 麻烦校对者也注意，在审核翻译的同时，也要审核格式问题。

@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@leviding
Copy link
Member

leviding commentedDec 4, 2019•
edited
Loading

麻烦两位@bemself@KerbalHao 都看下校对意见，现根据校对意见修改一遍，修改的同时需要检查是否还有类似问题，一并修改，辛苦了。

bemselfand others added7 commitsDecember 4, 2019 19:35
resolve review commentCo-Authored-By: LeviDing <imdingxuewen@gmail.com>
Co-Authored-By: LeviDing <imdingxuewen@gmail.com>
Co-Authored-By: LeviDing <imdingxuewen@gmail.com>
Co-Authored-By: LeviDing <imdingxuewen@gmail.com>
Co-Authored-By: LeviDing <imdingxuewen@gmail.com>
@bemself
Copy link
CollaboratorAuthor

@leviding 感谢校对，已修改~

/done

@bemself
Copy link
CollaboratorAuthor

@KerbalHao 感谢校对，已修改。

/done

@levidingleviding merged commit34f4904 intojavascript-tutorial:masterDec 5, 2019
@javascript-translate-bot

Error: the article already translated in the Progress Issue#324⁉️

@levidingleviding added DONE and removed needs +1 labelsDec 5, 2019
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@KerbalHaoKerbalHaoKerbalHao requested changes

@levidinglevidingleviding approved these changes

Assignees
No one assigned
Labels
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@bemself@KerbalHao@javascript-translate-bot@leviding
[8]ページ先頭
©2009-2025 Movatter.jp