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

Fix prop-types and create-react-class warning #56

Closed
wants to merge 2 commits into from

Conversation

ddcat1115
Copy link
Contributor

src/Dialog.tsx Outdated
@@ -49,7 +50,8 @@ function offset(el) {
return pos;
}

const Dialog = React.createClass<IDialogPropTypes, any>({
const Dialog = createReactClass<IDialogPropTypes, any>({
Copy link
Member

Choose a reason for hiding this comment

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

use es6 syntax ?

@paranoidjk
Copy link
Member

CI is broken

@paranoidjk
Copy link
Member

ci 看起来是 PhantomJS 不支持es6,@benjycui 看下测试

@benjycui
Copy link
Member

ci 看起来是 PhantomJS 不支持es6,@benjycui 看下测试

这不大可能吧,毕竟测试都会经过编译。

@benjycui
Copy link
Member

重新跑下 CI 吧,我没权限。

@afc163
Copy link
Member

afc163 commented Apr 27, 2017

ci 挂了

@afc163
Copy link
Member

afc163 commented May 7, 2017

@ddcat1115

@ddcat1115
Copy link
Contributor Author

由于某些复杂的原因。。。重新发了个一模一样的pr #59 这个关掉了

@ddcat1115 ddcat1115 closed this May 10, 2017
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.

None yet

4 participants