Спасиб)

@artalar Что скажешь про автогенерацию док? И у меня прямо в PR там пара вопросов, на которые я бы хотел получить ответ

@artalar Также с моим PR по поводу redux-compat. Как я уже сказал в PR, не обязательно накапливать функционал из разных мест, чтобы опубликовать этот пэкидж. Я сделал то, что сделал - и пэкидж можно выпустить, даже если там всего одна функция useStore. И даже если она без классных док.
Хороший принцип - все PR должны быть атомарны и быть про что-то одно (как в принципе SRP из SOLID). Мой PR - такой.
Я не люблю затягивать, а подход "сначала накопи достаточно функционала" может привести к тому, что redux-compat не будет выпущен ещё долго

Спасибо что пинганул.

В первую очередь стоит сказать что я тормоз 🙂

Начал смотреть ПР - мешают внутренние классы (Tree и т.п.) - все что в kernel не нужно документировать (точнее генерить доку по нему). Т.е. нужно больше @Internal навешать.

По поводу нейминга - можно поправить.

Итоговая верстка тоже странная получается - очень много места (пол+ экрана) знаимают заголовки и разделители без информативной нагрузки. Вообще вопрос как это лучше кастомайзить - я видел твой комментарий - а нет ли на просторах интернетов каких-то готовых “тем”?

Пакет на 1 функцию тоже оч странно и будет сбивать по началу.

Я в понедельник еще докину туда пару хелперов и выпущу.

Да навешать @internal - не проблема. Мне главное - чтобы мы согласовали  сам подход, и имеет ли смысл дальше продолжать.
По поводу заголовков и разделителей. Ты про шапку с логотипом реатома из readme.md? Если про это, то согласен, нужно это убрать, и сложностей тут нет.
По поводу готовых тем - я максимально использовал готовое и то, что уже юзается. Остальные варианты - гораздо хуже, потому что они не заточены под monorepo и API "не на классах" и выглядят вообще избыточно.

Я вот про это - для новичков это совсем не френдли

Можно это как-то вырезать?

Что именно будет сбивать? Тем более, что эта функция - реально полезная?
Про пэкидж этот вначале вообще никто знать не будет, особенно, пока в общую доку его не добавим.
Не обязательно ж сразу делать релиз npm-пакета, можно в отдельных PR добавить весь функционал, а потом зарелизить пакет

Да 👍👌

Мне тоже не очень нравится с такими сигнатурами. Кастомизировать это можно - но придётся делать форк генератора доки и темы оформления

Отлично!

@artalar В остальном с докой - норм? Как тебе JSDoc комменты в коде? По итогу они получаются достаточно длинными, особенно, если примеры добавлять. Лично мне - норм, потому что единый источник правды

Норм, но есть штуки которые очень хочется поправить, еще потыкаюсь и отпишусь

Спасибо, жду.

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

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

@artalar нашёл минус реатома при отладке. Невозможно регулировать отображение атомов в redux-devtools. Я бы там хотел видеть только скопанованные атомы, а получаю и исходные и скопанованные. У нас огромный проект это, и этот шум из атомов сильно мешает отладке.

тут нужно пилить свои девтулзы

тут нужно возвращаться к redux, а этого не хочется