gilbarbara / codeclimate-stylelint

A Code Climate engine for the mighty, modern CSS linter
24 stars 18 forks source link

Request: update Stylelint version #40

Closed efueger closed 4 years ago

efueger commented 4 years ago

Emily here from Code Climate Support.

With other plugins, this typically is a versioning issue.

@gilbarbara - what do you think? Do you think updating the plugin would help with this?

Let me know if I can provide additional details. Happy to help.

Thanks!

miktirr commented 4 years ago

Upgrading to the latest stylelint version (13.7.x) and configuration practices seems to break this engine.

This is because stylelint has deprecated *-blacklist, *-requirelist and *-whitelist rules in favor of the new *-disallowed-list, *-required-list and *-allowed-list ones.

If one migrates to the new rule names, this engine fails and throws error related to the new naming conventions:

Screen Shot 2020-09-07 at 13 22 02
efueger commented 4 years ago

Thanks for the report @miktirr .

Cheers, Emily

miktirr commented 4 years ago

Sure, @efueger! The repo itself is not public but you should see the build error here: https://codeclimate.com/repos/5da99cdb529bf4014e0092bb/builds/2165 I’m also happy to provide additional details of our setup, if it helps.

gilbarbara commented 4 years ago

hey @efueger You need to update the engine. It is running a really old version.

Anyway, it has been updated in 2.3.0 (engine 13.7.2)