search

alfa-laboratory / arui-presets-lint

A set of linting presets for arui-feather-based projects
14 stars 4 forks source link

feat(eslint): override rule for JSX-quotes #105

Closed SkripalBoris closed 4 years ago

SkripalBoris commented 4 years ago

Переопределение правила jsx-quotes

Мотивация и контекст

В базовом конфиге airbnb включены двойные кавычки для JSX атрибутов ссылка. Данные изменения устанавливают одинарные кавычки для JSX атрибутов, по аналогии со строками

voronin-ivan commented 4 years ago

Мотивация и контекст непонятны - чем ординарные кавычки для JSX аттрибутов будут лучше?

SkripalBoris commented 4 years ago

Мотивация и контекст непонятны - чем ординарные кавычки для JSX аттрибутов будут лучше?

До этого использовались одинарные кавычки. Кажется, что так выглядит единообразно

voronin-ivan commented 4 years ago

Мне кажется, что наоборот - в jsx мы описыванием подобие html, поэтому двойные кавычки выглядят уместнее. Выглядит, как вкусовщина, поэтому значение из airbnb я бы переопределять не стал.

SkripalBoris commented 4 years ago

Согласен, что это вкусовщина. Закрываю ПР