Как подружить destructuring assignment и eslint?

Question

[gracer]

Добрый день. Пытаюсь разобраться с eslint впервые. Использую VS Code, поставил всё что нужно для линта по туториалам, конфиг сгенерировал с помощью eslint --init.

module.exports = {
    "env": {
        "browser": true,
        "es6": true
    },
    "extends": [
        "airbnb-base"
    ],
    "globals": {
        "Atomics": "readonly",
        "SharedArrayBuffer": "readonly"
    },
    "parserOptions": {
        "ecmaVersion": 2018
    },
    "rules": {
    }
};

Начал исправлять ошибки в конфиге галпа и не понял зачем линтер ругается на

const { parallel, src, dest, watch } = require('gulp');

Expected a line break after this opening brace.eslint(object-curly-newline)

ведь это не объект. Подскажите как и стоит ли подружить destructuring assignment с eslint и что делать в подобных ситуациях, когда линтер ругается на вроде бы нормальную ситуацию.

Answer 1

[Леонид Князев]

В документации написано:

This rule enforces consistent line breaks inside braces of object literals or destructuring assignments.

Так, что в вашем случае надо писать:

const {
  parallel, src, dest, watch
} = require('gulp');

Comments

[gracer]

Если быть точнее, линтер требует такого (comma-dangle)

const {
  parallel, src, dest, watch,
} = require('gulp');

Это и есть те самые лучшие практики? Будет ли допустимо если изначальная запись выглядит нормально и просто отключить для этой строки правило?

// eslint-disable-next-line object-curly-newline
const { parallel, src, dest, watch } = require('gulp');

Мне просто интересно, что принято в мире. Всегда ли следуют правилам линтера или отключение правил в частных случаях распространено.

[Yustas Alexu]

gracer, забудьте про "принято в мире", некоторые даже не слышали про eslint. По поводу правил, все они описаны в документации и там же написано зачем нужно это правило. В частности запятая в конце нужна для гита, когда добавляется новая строка, запятая уже стоит. Если ее поставить при добавлении, то гит будет подсвечивать и строку с запятой как измененную. https://eslint.org/docs/rules/comma-dangle В общем правило не для всех. Если не нужно, можно отключить, как и любое другое правило.