LINT-86(Feedback): Refine bootstraping of linting

[Krakazybik]

Description

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

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

References

Checklist

• [x] Description added
• [x] Self-reviewed
• [x] CI pass

[azinit]

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

[Krakazybik]

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

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

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

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

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

https://user-images.githubusercontent.com/1334019/154853116-5374d7b6-1293-42ec-baad-937fd1b24f97.mp4

[azinit]

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

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

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

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

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

2022-02-20.19-28-53.mp4

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

[Krakazybik]

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

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

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

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

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

[Krakazybik]

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

https://user-images.githubusercontent.com/1334019/154857977-af631b71-8cc7-405d-a779-42c03075ac42.mp4

[Krakazybik]

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

[Krakazybik]

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

[azinit]

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

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

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

[Krakazybik]

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

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

[azinit]

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

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

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

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

[azinit]

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

[Krakazybik]

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