Skip to content

Remove Context calling React.Component constructor#612

Merged
justin808 merged 6 commits intomasterfrom
remove-unnecessary-param-basic-install
Nov 20, 2016
Merged

Remove Context calling React.Component constructor#612
justin808 merged 6 commits intomasterfrom
remove-unnecessary-param-basic-install

Conversation

@justin808
Copy link
Copy Markdown
Member

@justin808 justin808 commented Nov 20, 2016

The React.Component takes one param in a call to super in the
constructor.


This change is Reviewable

The React.Component takes one param in a call to super in the
constructor.
@justin808 justin808 force-pushed the remove-unnecessary-param-basic-install branch from b275918 to 90eb7f2 Compare November 20, 2016 01:28
@justin808
Copy link
Copy Markdown
Member Author

2016-11-19_15-30-53

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 82.987% when pulling 4cd4f87 on remove-unnecessary-param-basic-install into 540b51e on master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 82.987% when pulling 26897f1 on remove-unnecessary-param-basic-install into 540b51e on master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 82.987% when pulling 26897f1 on remove-unnecessary-param-basic-install into 540b51e on master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 82.987% when pulling 26897f1 on remove-unnecessary-param-basic-install into 540b51e on master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 82.987% when pulling 26897f1 on remove-unnecessary-param-basic-install into 540b51e on master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 82.987% when pulling 26897f1 on remove-unnecessary-param-basic-install into 540b51e on master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 82.987% when pulling 26897f1 on remove-unnecessary-param-basic-install into 540b51e on master.

@robwise
Copy link
Copy Markdown
Contributor

robwise commented Nov 20, 2016

lgtm

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.

3 participants