-
Notifications
You must be signed in to change notification settings - Fork 1k
Upgrade eslint + move to eslint-config-airbnb-base #1088
Conversation
e76a8df
to
d1284a0
Compare
This can be updated now after Prettier has been merged! |
Nice job here 😄 🚀 |
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.
Nice work here! I think you forgot to delete test/test-46-input-output-targets-many-6/test-output.exe-linux
260f4b2
to
334eb94
Compare
I have addressed all code review comments 🚀 |
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.
LGTM
Assuming the tests pass, let's merge! |
Tests are so slow, takes around 20/30 minutes :( |
It's important they pass with a change this large 😄 |
Yeah my comment was not about don't wait but just that we should work on that, them should take less then 10 minutes to improve development |
Totally agree. Luckily, they passed! |
Moving to standardized tooling and configs. Turned off rules that clashed with how the code is currently written.
This PR should be merged after #1076