codeclimate / codeclimate-eslint

Code Climate Engine for ESLint
MIT License
95 stars 93 forks source link

Add eslint 5 support #423

Closed noqcks closed 5 years ago

noqcks commented 5 years ago

I've upgrade to eslint 5 and upgraded some packages that support eslint5.

How has it been checked previously whether external dependencies support eslint version x? For example, for the eslint 4 branch? For me I checked whether eslint5 was in the dev dependencies for 3rd party dependencies, but this can produce some false negatives.

I've opened this PR up against the channel/eslint-4 branch because I am unable to create the channel/eslint-5 branch.

modosc commented 5 years ago

i'm happy to help if there's anything left to do here.

noqcks commented 5 years ago

Thanks @modosc just waiting on a review from someone on the codeclimate team.

We might have some unsupported dependencies in eslint5 that we'll have to remove.

cc: @efueger @wfleming

filipesperandio commented 5 years ago

@noqcks Sorry this is taking long. I've just pushed the channel/eslint-5 branch and updated to PR against it. I will do some testing next and let you know.

noqcks commented 5 years ago

no worries! @filipesperandio

thanks for taking a look 🙏

noqcks commented 5 years ago

@filipesperandio I remember the reason why I removed eslint-config-secure

CI is throwing the error I was getting locally

error An unexpected error occurred: "could not find a copy of eslint to link in /usr/local/node_modules/eslint-config-secure/node_modules".
noqcks commented 5 years ago

Thanks @filipesperandio !!

If all goes well, how do I know once eslint5 becomes available on the codeclimate platform? Keep an eye on the changelog?

filipesperandio commented 5 years ago

Will be updating that soon, though, I've just merged PRs enabling it, so the channel should be available soon.