Вообще мои имховые правила:

Имена файлов - kebab-case.
Все импорты только по относительным путям.
Классы - PascalCase, функции и переменные - camelCase

Абревиатуры в именах не капсом: encodedUrl, а не encodedURL

Короткие имена разрешены для локальных переменных в коротких предикатах:
products.filter(p => p.hasSkus)

Разрешается абревиировать имена, если в контексте понятно, что имя означает. Типа const xhr = new XMLHttpRequest()

А чего относительные пути а не абсолютные? Становится ведь сильно неудобно выносить разные функции и потом читать, откуда конкретный объект импортили

> Короткие имена разрешены для локальных переменных в коротких предикатах:
> products.filter(p => p.hasSkus)

THIS! как же меня триггерит от излишних деструктуризаций типа

products.filter(({ hasSkus }) => hasSkus)

products.filter(({ hasSkus }) => hasSkus)

А еще

products.filter(product => product.hasSkus)

Потом лови опечатки, где ты product, а где products написал.

Плюс за этим лесом из products product product вообще не видно, что блин вообще в коде делается.

Джавашня.

дякую

1. Тебе похрен откуда функция - ты cmd-click - и попал туда, где она объявлена

2. Любая тулза понимает относительные пути. Тебе не надо настраивать какие-то волшебные переменные среды / настройки проекта, которые понимает, например Вебпак, но не понимает какой-нибудь линтер или какая-то среда разработки. Жизнь слишком коротка, чтоб заниматься такой херней.

про жизнь коротка согласен

Файлы кебаб-кесом, потому что люди сидят и на case-sensitive File Sytems, и на Case Insensitive. Все буквы маленькие - и весь этот кросплатформенный гемор пройдет мимо тебя.

Ну и тогда вопрос между - и _ в именах. Имена файлов - еще и урлы через file:/ или даже через http:// если я их сервлю каким-то сервером статическим. Подчеркивания в урлах смотрятся говенно (смотри урлы Википедии для примера). Поэтому kebab-case - самый разумный выбор

products.filter(({ hasSkus }) => hasSkus)

Чего триггерит-то?

можно ещё весь код в одной папке держать, тогда относительные пути вообще не создают никаких проблем

у каждой фичи есть стоимость. мне кажется эта фича слишком дорогая для такого маленького случая. попросту с ней получается, посимвольно, больше кода, а если нужно расширить условие то нужно вытягивать новые ключи. я просто не понимаю зачем лишний раз страдать

Ну я к тому, что мне на ревью как-то комфортнее читать import ololo from types/someDomain, чем ../../../someDomain

я же и говорю, просто весь код в одной папке храни)

Я на ревью не читаю импорты. Зачем?

о, вот кстати чтобы я ещё дописал в бест практицес это “не создавать папки по типу кода: reducers, components, hooks, …”

+100 к Леше

«так, вот тут у нас теперь новый набор редьюсеров, он будет тянуться с такого файла»

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

Ладно, вот это уже неплохо. Ты за «все внутри одного модуля компонента»?