LINT-86(Feedback): Refine bootstraping of linting

[Krakazybik]

Description

+1400 из pnpm-lock файла

Пока что драфт, с кучей косяков на посмотреть.

References

Checklist

• Description added
• Self-reviewed
• CI pass

[azinit]

PRO-TIP: Чтобы люди не шугались диффа, лучше заранее подсвечивать что это из-за yarn.lock или еще чего-то другого, что можно скипнуть при ревью

[azinit]

Пока выглядит чет как хоть и "проработанная" но "жирная" реализация для простого npm install))

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

Пока что же крутится мысль "Блин. А нельзя никак упростить это все? :DD"

[Krakazybik]

Пока выглядит чет как хоть и "проработанная" но "жирная" реализация для простого npm install))

А нам надо что? Не понимаю тогда к чему все диолги, о том что нам слишком много кнопок надо нажать чтоб установить.

Пока что же крутится мысль "Блин. А нельзя никак упростить это все? :DD"

Упростить в плане?

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

2022-02-20.19-28-53.mp4

[azinit]

А нам надо что? Не понимаю тогда к чему все диолги, о том что нам слишком много кнопок надо нажать чтоб установить.

Упростить в плане?

Ну просто у меня в голове было что-то простое на уровне простого bash-скрипта не больше 100 строчек))

А тут и роллап, и несколько доп.модулей для работы скрипта, и pnpm еще зачем-то... (хотя ясно что можно было и без pnpm)

Поэтому и говорю, что надо пересмотреть еще раз требования по задаче и понять - точно ли никак это проще нельзя было сделать 🤔
(это не для обесценивания твоих стараний, а чтобы нам же меньше потом проблем по поддержке и стабилизации этого было)

2022-02-20.19-28-53.mp4

Но выглядит красиво красиво конечно, тут ничего не попишешь))

[Krakazybik]

Ну просто у меня в голове было что-то простое на уровне простого bash-скрипта не больше 100 строчек))

Ога, для виндовс. Bash скрип. 🤣 (p.s. я понимаю что ты про js)) А потом мы захотим баш скриптом патчить конфиг, или выбирать нужные опции конфига, и давай теперь новый баш скрипт выдумывать. А здесь всё в принципе расширяемо. На любом этапе можно выполнить любую команду добавив нужный HOF, для патчинга, добавь строчку в questions и тд.

Полюбому опять будет кому-то сложно и он скажет я не хочу дописывать lite/experemental, пусть оно само делает.

А тут и роллап, и несколько доп.модулей для работы скрипта, и pnpm еще зачем-то... (хотя ясно что можно было и без pnpm)

Это из расчета на то, что ты утащишь в отдельный пакет, т.к. зачем нам зависимости лишние для eslint-config?

[Krakazybik]

Мона вывод инфы оставить

2022-02-20.21-25-46.mp4

[Krakazybik]

Мона это в принципе слепить в одну команду )) Но тогда не так эффекто смотреться будет xD

[Krakazybik]

Пока слепил в js и ts отдельно

[azinit]

она вывод инфы оставить

Пересмотрел оба скринкаста - но не увидел разницы, честно честно)))

Или то что во "втором" еще показывается как устанавливается каждая зависимость отдельно?

[azinit]

В целом норм, хотя по имплементации местами хочется прям придираться если сильно вчитываться

Но в рамках задачи некрит)

Самое главное - что довел до конца и не выгорел (надеюсь) пока писал
Остальное - разберемся)

Еще раз спс, что с усердностью довел PR до конца и описал ридмиху!
(хотя надеюсь оба уже понимаем, что драфты лучше не кидать для ревью 😏 )

[Krakazybik]

Пересмотрел оба скринкаста - но не увидел разницы, честно честно)))
Или то что во "втором" еще показывается как устанавливается каждая зависимость отдельно?

Ога, типо красивенькое.

[azinit]

Пересмотрел оба скринкаста - но не увидел разницы, честно честно)))
Или то что во "втором" еще показывается как устанавливается каждая зависимость отдельно?

Ога, типо красивенькое.

Ну чисто субъективно - мне больше нравится более минималистичный варик, где меньше лишней инфы

Но если оч хочешь - то можно под каким-нибудь флагом --verbose чтобы выводилось с логами такими)
(а по дефолту - коротенько, как на первом скринкасте)

[azinit]

@Krakazybik Влей как сочтешь нужным)
(из критов прям - pnpm-lock для cli разве что - но там если что сам его удалю наверн)

[Krakazybik]

Ну чисто субъективно - мне больше нравится более минималистичный варик, где меньше лишней инфы
Плюсик.

[azinit]

Ну красотень прям)

Заметь, с yarn и кол-во строк снизилось 😏