Conversation
a0bf4b1
to
1cf2578
Compare
1cf2578
to
e8c5792
Compare
e8c5792
to
fbc1c5f
Compare
b740cf3
01e579f
to
2635f59
Compare
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
|
||
- name: redux | ||
repo: https://github.com/reactjs/redux | ||
commit: ac7cd32a5270eb374611bf73d072e84a1e7567c2 | ||
commit: 9819b5ef2909cacda0f09c8d29fde229bd7da5fa |
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.
Updated to ESLint v6 here: reduxjs/redux#3698
@@ -25,60 +24,42 @@ | |||
|
|||
- name: mocha | |||
repo: https://github.com/mochajs/mocha | |||
commit: 09ce746aa925d35317f2624fd36c77a31bb68e24 | |||
commit: 883ae4b5e1aacd8cf30694da33b391ce58f4cca8 |
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.
Updated to ESLint v6 here: mochajs/mocha#4138
projects.yml
Outdated
- --rule=indent:off | ||
- --rule=no-multi-spaces:[error,{ignoreEOLComments:true}] | ||
- --rule=camelcase:off # There are several `camelcase` errors in the vue codebase for some reason | ||
# - name: vue |
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.
Waiting on vuejs/vue#11099 to land before we can merge this.
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.
I decided to disable the failing rules and we can update later on.
b6c8d89
to
453be31
Compare
- test | ||
- examples | ||
- --ext js,ts | ||
- src/**/* |
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.
This fails without specifying this glob.
This is ready to go! |
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. Nice job with all the yak shaving needed to land this.
Small PR to update dependencies and drop support for Node < 10.
Does this seem okay? We could keep support for Node 8, but since it will be out of maintenance mode soon I don't think it's worth it.