-
Notifications
You must be signed in to change notification settings - Fork 809
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
Convert library code to ES2015+ #286
Comments
@claydiffrient Already progressing? |
@abouthiroppy I haven't started on the library code at this point. I'm working on getting the tests completed first as that should be easier. |
I see. Thanks. I would like to cooperate🙃 |
@abouthiroppy I ended up accidentally getting things converted over as part of another issue I was working on. If you'd like to review 0b4ab7d that'd be awesome though :) It's PR #293 |
@claydiffrient Thank you for telling me :) Great work!!! |
* Add linting * Make specs pass linter * Make lib/helpers pass linter * Make lib/components pass linter This also does some signfiicant refactoring of the code to appease the linter's pro-ES2015+ stance. closes reactjs#289 closes reactjs#286 * Make travis run the lint task as well as specs closes reactjs#284
* Add linting * Make specs pass linter * Make lib/helpers pass linter * Make lib/components pass linter This also does some signfiicant refactoring of the code to appease the linter's pro-ES2015+ stance. closes reactjs#289 closes reactjs#286 * Make travis run the lint task as well as specs closes reactjs#284
* Add linting * Make specs pass linter * Make lib/helpers pass linter * Make lib/components pass linter This also does some signfiicant refactoring of the code to appease the linter's pro-ES2015+ stance. closes reactjs#289 closes reactjs#286 * Make travis run the lint task as well as specs closes reactjs#284
This would provide a much better developer experience going forward.
The text was updated successfully, but these errors were encountered: