-
Notifications
You must be signed in to change notification settings - Fork 664
[zh-cn] synced from #de89293 to #a5ad66a #168
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
Conversation
import sinon from 'sinon' | ||
|
||
describe('点击事件', () => { | ||
it('在 yes 按钮上点击会调用我们的方法并附带参数 "yes"', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个翻译么
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
其它几个页面的我都翻译了,也可以保留,翻译的时候确实有点纠结
Btw,我纠结的地方还不止这一处,可以待我稍后整理一下然后集中起来再讨论
@@ -46,15 +46,15 @@ npm install --save-dev jest-vue | |||
"vue" | |||
], | |||
"transform": { | |||
// 用 `jest-vue` 处理 `*.vue` 文件 | |||
".*\\.(vue)$": "<rootDir>/node_modules/jest-vue" | |||
// 用 `vue-jest` 处理 `*.vue` 文件 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个 JSON 不合法耶……给原文提 issue 么?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
恩,我来提一下好了
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
不过看了下别的地方还有不少这样的
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
我也发现了
@yyx990803 我留意到这部分英文文档正好是你提交的 f68e1ed 方便拿个主意吗?是否担心这会误导开发者在 package.json
里写 JS 注释导致出错?
谢谢
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
想想应该不太需要……连这个都搞不请估计也不会写测试
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
应该没关系,留着就行
Hey guys, if a native speaker could confirm these changes I'll merge |
ref: de89293...a5ad66a
The main updates is
guides/dom-events
,api/wrapper/destroy
andapi/wrapper-array/destroy
./ping @Justineo @fnlctrl
Thanks.