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

将 JavaScript 放到一边 —— 用 Mint 这门强大的语言来创建一个 Web 应用 #8521

Conversation

Usualminds
Copy link
Contributor

译文翻译完成,resolve #8491

WebTransport 会在不久的将来取代 WebRTC 吗?
WebTransport 会在不久的将来取代 WebRTC 吗?
will-webtransport-replace-webrtc-in-near-future
leave-javascript-aside-mint-is-a-great-language-for-building-web-apps
将 JavaScript 放到一边 —— 用 Mint 这门强大的语言来创建一个 Web 应用
@Usualminds Usualminds changed the title Leave javascript aside mint is a great language for building web apps 将 JavaScript 放到一边 —— 用 Mint 这门强大的语言来创建一个 Web 应用 Apr 29, 2021
@Usualminds
Copy link
Contributor Author

@lsvih 翻译完成

@KimYangOfCat
Copy link
Contributor

@lsvih 校对认领

@lsvih
Copy link
Member

lsvih commented Apr 29, 2021

@KimYangOfCat 欢迎校对

Copy link
Contributor

@KimYangOfCat KimYangOfCat left a comment

Choose a reason for hiding this comment

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

@lsvih 校对完毕
@Usualminds 赞,译文整体翻译质量很高哒!
不过有些地方的意译得有些简略了,丢失了部分原文信息。

@KimYangOfCat
Copy link
Contributor

@Usualminds 对了,还有一个问题希望你能关注一下,你的这个 Pr 中 有一些 Commit 的内容是与本文的翻译无关的,建议你按照严格的 github Pr 流程来提交 Pr,尽量避免这种冗余的 commit 内容。这样会使仓库的 Commit 节点清晰且简洁,后续不会造成一些莫名其妙的冲突。

@Chorer
Copy link
Contributor

Chorer commented Apr 30, 2021

校对认领

Copy link
Contributor

@Chorer Chorer left a comment

Choose a reason for hiding this comment

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

校对完毕

@Usualminds
Copy link
Contributor Author

@Usualminds 对了,还有一个问题希望你能关注一下,你的这个 Pr 中 有一些 Commit 的内容是与本文的翻译无关的,建议你按照严格的 github Pr 流程来提交 Pr,尽量避免这种冗余的 commit 内容。这样会使仓库的 Commit 节点清晰且简洁,后续不会造成一些莫名其妙的冲突。

get

@Usualminds
Copy link
Contributor Author

@lsvih 校对意见已合入
@KimYangOfCat @Chorer @PassionPenguin 感谢三位的校对,校对意见已合入

@PassionPenguin
Copy link
Contributor

@Usualminds 对了,还有一个问题希望你能关注一下,你的这个 Pr 中 有一些 Commit 的内容是与本文的翻译无关的,建议你按照严格的 github Pr 流程来提交 Pr,尽量避免这种冗余的 commit 内容。这样会使仓库的 Commit 节点清晰且简洁,后续不会造成一些莫名其妙的冲突。

get

不能说是不是无关的commit,而且这个问题很多人都会有,也没有特别完美的解决方案。目前 best practices 应该是:

cd ./gold-miner
# define upstream to xitu/gold-miner
# git remote add upstream git@github.com:xitu/gold-miner.git
git fetch upstream
git checkout master
# force reset
git reset --hard upstream/master
# force push
git push --force

@KimYangOfCat
Copy link
Contributor

@Usualminds 对了,还有一个问题希望你能关注一下,你的这个 Pr 中 有一些 Commit 的内容是与本文的翻译无关的,建议你按照严格的 github Pr 流程来提交 Pr,尽量避免这种冗余的 commit 内容。这样会使仓库的 Commit 节点清晰且简洁,后续不会造成一些莫名其妙的冲突。

get

不能说是不是无关的commit,而且这个问题很多人都会有,也没有特别完美的解决方案。目前 best practices 应该是:

cd ./gold-miner
# define upstream to xitu/gold-miner
# git remote add upstream git@github.com:xitu/gold-miner.git
git fetch upstream
git checkout master
# force reset
git reset --hard upstream/master
# force push
git push --force

@PassionPenguin
wiki 中,有非常清晰的步骤,其中有一点很关键的步骤是 4.新建分支,通过新建分支以及严格遵守当前分支只修改与当前分支有关的内容,就能有效隔绝在 PR 中提交无关的 commit 内容,我之前遇到过类似的问题,你的方案中缺少重要的新建分支的步骤。

是不是无关的commit 通过查看 commit 的 files change 应该能很直观的呈现出来。并不是没有完美的解决方案,可能只是你目前不了解,不过完美解决需要重新提交 PR。这个问题很多人有并不代表它不值得注意,还有我只是提议楼主一下而已,在翻译计划里面并没有对这个做硬性规定,所以修不修改,无所谓的。至于具体要不要添加这个规定,只能另开 issue 讨论。

查看了楼主的仓库之后,发现他其实是有新建分支的,可能只是在修改内容时没有注意当前分支应该只修改与当前分支有关的内容吧。还有其中有非常多的 commit 其实都是通过 github 反向 PR 时产生的,这是使用 github PR 更新个人仓库的一大弊端。但是在 wiki 中又并没有对后续更新个人仓库这一操作做出具体的描述。我个人认为可以参考印记中文 webpack 翻译流程中的相关描述。

参考:

  • 我之前经历的 PR,不过其中关键的 commit 信息,已经由于我git push -f 操作被覆盖掉了,已经丢失了。

@lsvih
Copy link
Member

lsvih commented May 1, 2021

回头更新一下 wiki 关于同步 fork 相关内容哈,感谢大家的讨论

@lsvih lsvih merged commit 838c93e into xitu:master May 1, 2021
@lsvih
Copy link
Member

lsvih commented May 1, 2021

@Usualminds 已经 merge 啦~ 快快麻溜发布到掘金然后给我发下链接,方便及时添加积分哟。

掘金翻译计划有自己的知乎专栏,你也可以投稿哈,推荐使用一个好用的插件
专栏地址:https://zhuanlan.zhihu.com/juejinfanyi

@Usualminds
Copy link
Contributor Author

Usualminds commented May 2, 2021

@PassionPenguin
Copy link
Contributor

@Usualminds 对了,还有一个问题希望你能关注一下,你的这个 Pr 中 有一些 Commit 的内容是与本文的翻译无关的,建议你按照严格的 github Pr 流程来提交 Pr,尽量避免这种冗余的 commit 内容。这样会使仓库的 Commit 节点清晰且简洁,后续不会造成一些莫名其妙的冲突。

get

不能说是不是无关的commit,而且这个问题很多人都会有,也没有特别完美的解决方案。目前 best practices 应该是:

cd ./gold-miner
# define upstream to xitu/gold-miner
# git remote add upstream git@github.com:xitu/gold-miner.git
git fetch upstream
git checkout master
# force reset
git reset --hard upstream/master
# force push
git push --force

@PassionPenguin
wiki 中,有非常清晰的步骤,其中有一点很关键的步骤是 4.新建分支,通过新建分支以及严格遵守当前分支只修改与当前分支有关的内容,就能有效隔绝在 PR 中提交无关的 commit 内容,我之前遇到过类似的问题,你的方案中缺少重要的新建分支的步骤。

是不是无关的commit 通过查看 commit 的 files change 应该能很直观的呈现出来。并不是没有完美的解决方案,可能只是你目前不了解,不过完美解决需要重新提交 PR。这个问题很多人有并不代表它不值得注意,还有我只是提议楼主一下而已,在翻译计划里面并没有对这个做硬性规定,所以修不修改,无所谓的。至于具体要不要添加这个规定,只能另开 issue 讨论。

查看了楼主的仓库之后,发现他其实是有新建分支的,可能只是在修改内容时没有注意当前分支应该只修改与当前分支有关的内容吧。还有其中有非常多的 commit 其实都是通过 github 反向 PR 时产生的,这是使用 github PR 更新个人仓库的一大弊端。但是在 wiki 中又并没有对后续更新个人仓库这一操作做出具体的描述。我个人认为可以参考印记中文 webpack 翻译流程中的相关描述。

参考:

  • 我之前经历的 PR,不过其中关键的 commit 信息,已经由于我git push -f 操作被覆盖掉了,已经丢失了。

你的回答难以让人信服,因为commit file changes 最终也只是当前的文件。每一次翻译都选择再次克隆,大抵是没考虑大陆用户的网速。你所提出的意见与可行也并不比我的好

@PassionPenguin
Copy link
Contributor

@Usualminds 对了,还有一个问题希望你能关注一下,你的这个 Pr 中 有一些 Commit 的内容是与本文的翻译无关的,建议你按照严格的 github Pr 流程来提交 Pr,尽量避免这种冗余的 commit 内容。这样会使仓库的 Commit 节点清晰且简洁,后续不会造成一些莫名其妙的冲突。

get

不能说是不是无关的commit,而且这个问题很多人都会有,也没有特别完美的解决方案。目前 best practices 应该是:

cd ./gold-miner
# define upstream to xitu/gold-miner
# git remote add upstream git@github.com:xitu/gold-miner.git
git fetch upstream
git checkout master
# force reset
git reset --hard upstream/master
# force push
git push --force

当然如果当前分支没有被污染,直接 pull 这个仓库也是可以的

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

将 JavaScript 放到一边 —— 用 Mint 这门强大的语言来创建一个 Web 应用
5 participants