Few bugs caught by ESLint. It fixes a couple of bugs where the use of an undefined variable would have crashed all the JS.

CommentFileSizeAuthor
core-js-eslint-errors.patch12.39 KBnod_

Comments

nod_’s picture

nod_
Primary language French
Location Lille
commented

After the patch, both JSHint and ESLint validates.

Anonymous’s picture

Anonymous (not verified) commented

I can confirm that this fixes a number of eslint errors (variable scope, trailing comma, undefined vars) and both validate.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Reviewed & tested by the community
yesct’s picture

yesct commented
Issue tags: +Coding standards
alexpott’s picture

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

Committed 59fee19 and pushed to 8.x. Thanks!

  • Commit 59fee19 on 8.x by alexpott: 
    Issue #2264241 by _nod: Fix ESLint errors and fix js bugs.

Status: Fixed » Closed (fixed)

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