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

Next #57

Closed
wants to merge 4 commits into from
Closed

Next #57

wants to merge 4 commits into from

Conversation

KillWolfVlad
Copy link
Contributor

@KillWolfVlad KillWolfVlad commented Sep 3, 2023

Fix #38
Fix #52
Fix #46

BREAKING CHANGE:  bump eslint plugins for major versions
Copy link
Member

@sadcitizen sadcitizen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Зачем надо было менять npm на yarn?

@@ -0,0 +1,5 @@
trailingComma: all
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Забери настройки форматирования из старого файла .prettierrc.

Copy link
Contributor Author

@KillWolfVlad KillWolfVlad Sep 4, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Все наши OpenSource проекты должны следовать единым стандартам форматирования, которые заданы в https://github.com/Byndyusoft/node-base-template и использовать одинаковый пакетный менеджер - yarn

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я не участвовал в разработке этих стандартов. Они нелегитимны и не могут применяться в этому пакету.

@@ -57,3 +62,7 @@ Note: If you have React project created with create-react-app, then you haven't
- [@KillWolfVlad](https://github.com/KillWolfVlad)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Забавная сортировка мейнтейнеров.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Сортировку мейнтейнеров не менял


[*]
indent_style = space
indent_size = 2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

4

@@ -1,112 +1,128 @@
/*
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Отличный пример того, как форматирование сделало кодревью максимально сложными.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Форматирование поменялось только в 3a67c0a, никаких изменений в правилах в этом коммите нет.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Комментарий был написан к PR, а не к коммиту

@sadcitizen sadcitizen closed this Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants