Problem/Motivation

$ eslint core/modules/quickedit/js/editors/ --quiet

core/modules/quickedit/js/editors/formEditor.js
   99:11  error  Expected indentation of 8 characters but found 10  indent
  100:11  error  Expected indentation of 8 characters but found 10  indent
  101:10  error  Expected indentation of 6 characters but found 8   indent

✖ 3 problems (3 errors, 0 warnings)

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
eslint-quickedit-formEditor.js_.patch693 bytesolli

Comments

olli created an issue. See original summary.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Status: Needs review » Reviewed & tested by the community

Thanks!

Status: Reviewed & tested by the community » Needs work

The last submitted patch, eslint-quickedit-formEditor.js_.patch, failed testing.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Status: Needs work » Reviewed & tested by the community

Random fails due to #2552687: Test failures in ConfigFormOverrideTest and ContainerRebuildWebTest on newly spun up testbot instances.

Wim Leers queued eslint-quickedit-formEditor.js_.patch for re-testing.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed 46ffe70 and pushed to 8.0.x. Thanks!

  • alexpott committed 46ffe70 on 8.0.x 
    Issue #2552831 by olli: Eslint errors in quickedit/js/editors/formEditor...

Status: Fixed » Closed (fixed)

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