search

anmedio / junost

🌱 Объективные грейды для развития специалистов сферы digital, предложенные и развиваемые профессиональным сообществом.
https://anmedio.github.io/junost/
110 stars 26 forks source link

Tooling, formatting and code style #27

Open norskeld opened 4 years ago

norskeld commented 4 years ago

Привет! Спасибо за проект, полезная инициатива. Однако, думаю, есть что доработать, раз уж проект в open source и предполагает участие совершенно посторонних людей.

  1. Prettier есть в зависимостях и для него даже имеется некое подобие конфига, но он не фигурирует ни в одном npm-скрипте и, судя по коду, его никогда не запускали. 😁Также используется eslint, но при этом не используется lint-staged, чтобы линтить и форматировать staged файлы на стадии коммита. Как результат: image Нехорошо. На мой взгляд, lint-staged сегодня must have для любого JS/TS проекта, тем более настраивается он за пару минут.

  2. Хорошо бы все-таки настроить prettier, явно указав настройки в .prettierrc и продублировав некоторые из них в .editorconfig. Например, те же размер и стиль индентации, final newline. Мелочь, а приятно и иногда полезно, тем более editorconfig подхватывается WebStorm'ом автоматически, и во многих редакторах при наличии плагина также подтягиваются настройки.

  3. Ничего не имею против gulp, я им легаси на работе транспайлю и собираю, но зачем он, когда рядом лежит webpack? Вообще, наверное, можно было и CRA воспользоваться изначально.

  4. Makefile, запускающий npm-скрипты. 🤔

NikolasMelui commented 4 years ago

Привет, спасибо за внимание к проекту! 😃 Отличные предложения, мы с радостью рассмотрим (и, скорее всего, примем) ваши PR на любую из вышеописанных тем, если вы, конечно, найдёте свободное время и сделаете эти самые PR 🙂 А по поводу Makefile, на проект заходят люди, которые понятия не имеют, что такое npm и как им пользоваться, однако давно знакомы с Makefile и сразу могут запустить проект 🙂

norskeld commented 4 years ago

Нет проблем, займусь. 👌Там еще и целая гора очепяток в данных по грейдам...

NikolasMelui commented 4 years ago

Можно глобально это всё зарефакторить, продумать архитектуру распределения скиллов, взять CRA и пересадить всё "на чистый лист". Можете написать мне на cto@anmedio.ru и мы обсудим это подробнее.

norskeld commented 4 years ago

Я, к сожалению, с реактом не очень дружу, и на глобальные вещи у меня время вряд ли найдется. Но с тулингом и текстами помогу. Их вы так или иначе переиспользуете, если будете рефакторить. 😀

NikolasMelui commented 4 years ago

Тогда договорились, ждём ваших PR 🙂