-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Transform variables #1277
Transform variables #1277
Conversation
Was this done manually, or solely with |
First I ran |
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.
Thanks!
Moving forward it would be preferable to put all of the automated changes in their own commit and follow up with commits that contain only manual changes. That would make it easier to review.
Okay, I'll definitely do that in the future! Thanks for guiding me along this, I don't have much experience in open source :) |
You are doing a great job! Keep it up 💃 |
In #1277 we enabled some new eslint rules, but the branch hadn't been freshly rebased, so a few failures snuck in. I fixed these by running ```sh npm run lint -- --fix ```
Continue work on #1240.
Transformed
var
statements toconst
andlet
, added eslint rules to enforce them.Not included:
If there is anything I can do to make these large scale changes easier to handle, please let me know!