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

Do Lint Better #1002

Merged
merged 23 commits into from
Apr 24, 2019
Merged

Do Lint Better #1002

merged 23 commits into from
Apr 24, 2019

Conversation

drewpc
Copy link
Collaborator

@drewpc drewpc commented Apr 23, 2019

This addresses all of #999 and incorporates/relies on changes in #997 and #1001 to be approved/integrated. Things shouldn't be linked together, I know, but it's hard to not do that with the number of changes coming.

@drewpc drewpc added this to the 1.0 milestone Apr 23, 2019
drewpc added 3 commits April 23, 2019 10:37
…int is the standard. Incorporate the React ESlint rules as well.
…ues with Acorn. Also...make eslint 4 super super happy happy!
@drewpc drewpc requested review from doug-wade, gigabo and sresant and removed request for doug-wade and gigabo April 24, 2019 15:30
@drewpc drewpc requested a review from roblg April 24, 2019 15:30
@drewpc
Copy link
Collaborator Author

drewpc commented Apr 24, 2019

Since this is a minor infrastructure change and all tests are passing, I'll merge after one approval.

drewpc added 3 commits April 24, 2019 14:10
…int is the standard. Incorporate the React ESlint rules as well.
…ues with Acorn. Also...make eslint 4 super super happy happy!
@drewpc drewpc merged commit 80dca46 into redfin:master Apr 24, 2019
@drewpc drewpc deleted the upgrade-lerna-lint-better branch April 24, 2019 20:48
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