Keeping up to date with new rules additions to eslint.

We can now check indentation with eslint. Which is what this patch does.

CommentFileSizeAuthor
core-js-eslint-rules-0.14.1.patch1.26 KBnod_
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾Seville 💃, Andalusia 🇪🇸, UTC+2 🇪🇺
CreditAttribution: penyaskito commented
Status: Needs review » Reviewed & tested by the community

I've been failing on this at #2422017: Split drupal.js. Applying this patch makes eslint show me the warnings on my previous patches, which were not caught before.

Wim Leers’s picture

Wim Leers
Location Ghent 🇧🇪🇪🇺
CreditAttribution: Wim Leers commented
Issue tags: +JavaScript, +Quickfix, +DX (Developer Experience)

Hurray! :)

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Yay coding standards.

Committed and pushed to 8.0.x. Thanks!

  • webchick committed 097240a on 8.0.x 
    Issue #2428041 by nod_: Update eslint config for 0.14.1

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.