Skip to content

Fix: Replace var with let and const in documentation #8051

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

Merged
merged 1 commit into from
Oct 22, 2016

Conversation

OMantere
Copy link
Contributor

@OMantere OMantere commented Oct 22, 2016

This was mentioned in #8035

Updates var variable declarations in the documentation examples to use ES6-style const, let and import, except for the "React without ES6" page and the LinkedStateMixin example on "Two way binding helpers", as mixins are not supported in ES6.

I could also update the CodePen examples on "Thinking in React" that still use var, can I link my own fork into the docs?

@facebook-github-bot
Copy link

Thank you for your pull request. We require contributors to sign our Contributor License Agreement, and yours has expired.

Before we can review or merge your code, we need you to email [email protected] with your details so we can update your status.

@gaearon
Copy link
Collaborator

gaearon commented Oct 22, 2016

Thanks!

gaearon pushed a commit that referenced this pull request Oct 22, 2016
acusti pushed a commit to brandcast/react that referenced this pull request Mar 15, 2017
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.

3 participants