Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tooling, formatting and code style #27

Open
norskeld opened this issue Jan 23, 2020 · 5 comments
Open

Tooling, formatting and code style #27

norskeld opened this issue Jan 23, 2020 · 5 comments

Comments

@norskeld
Copy link

Привет! Спасибо за проект, полезная инициатива. Однако, думаю, есть что доработать, раз уж проект в 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
Copy link
Contributor

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

@norskeld
Copy link
Author

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

@NikolasMelui
Copy link
Contributor

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

@norskeld
Copy link
Author

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

@NikolasMelui
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants