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

Не SOLIDно #16

Closed
leemuar opened this issue Dec 1, 2017 · 26 comments
Closed

Не SOLIDно #16

leemuar opened this issue Dec 1, 2017 · 26 comments

Comments

@leemuar
Copy link

leemuar commented Dec 1, 2017

Библиотека не соответствует SOLID. Из-за этого ее труднее расширять, добавлять новые транспорты.
Переписать как менеджер сообщений с возможностью подключения любого количества транспортов на лету.
Реализовать библиотеку распространенных транспортов.
Реализовать #8 как стандартный транспорт библиотеки

@theshadowco
Copy link
Collaborator

Если честно, это уже сделано https://github.com/bia-tech/messenger

@leemuar
Copy link
Author

leemuar commented Dec 1, 2017

Надо релизить!

@theshadowco
Copy link
Collaborator

Поломается же все у вас :)

@leemuar
Copy link
Author

leemuar commented Dec 1, 2017

Ну новым репозиторием тогда, этот deprecate'нуть

@theshadowco
Copy link
Collaborator

тоже вариант.
@nixel2007 @EvilBeaver что скажете?

@leemuar
Copy link
Author

leemuar commented Dec 1, 2017

Вообще эта библиотека имеет смысл, если есть logos? Он тоже отправляет сообщения, имеет весь нужный функционал, расширяем

@theshadowco
Copy link
Collaborator

Назначение все-таки разное, в notepad++ тож можно разрабатывать, но IDE как то больше подходит

@leemuar
Copy link
Author

leemuar commented Dec 1, 2017

Почему разное-то? Отправить куда-то сообщение, настроить способы отправки

@theshadowco
Copy link
Collaborator

logos
Логирование в стиле log4j

Библиотека предназначена для удобного вывода сообщений в привязке к их "уровням важности"

Функционал logos не повторяет полностью log4j, а, скорее, берет из него какие-то аспекты поведения.

@leemuar
Copy link
Author

leemuar commented Dec 1, 2017

Суть-то одинаковая - отправить сообщение. В транспорте игнорируй уровень важности - и все

@nixel2007
Copy link
Member

В транспорте игнорируй уровень важности - и все

Кто там про солид говорил? На мой взгляд, несмотря на кажущуюся одинаковость библиотек, они все же про разное.

По брейкинг чейндж - если совсем уж апи не сохранить, то можно релизнуть 2.0. Семвер, все дела, кто обновился без проверки - ССЗБ.

@nixel2007
Copy link
Member

Идея со старым классом в виде v1 отличная

@leemuar
Copy link
Author

leemuar commented Dec 2, 2017

Семвер, все дела, кто обновился без проверки - ССЗБ.

Воу-воу, зачем так кардинально-то. Полной работы с зависимостями и семвером в опм же пока нет (всякие там ^), может лучше все же новой библиотекой это выпустить?

@artbear
Copy link
Member

artbear commented Dec 2, 2017

новая версия в формате v2 отлично.
не нужно умножать сущности.
разные библиотеки с очень похожим фунционалом, фу-фу

@leemuar
Copy link
Author

leemuar commented Dec 4, 2017

разные библиотеки с очень похожим фунционалом, фу-фу

https://github.com/silverbulleters/vanessa-behavior
https://github.com/silverbulleters/vanessa-behavior2

¯\ _ (ツ) _ /¯

@nixel2007
Copy link
Member

Vb2 сделан с целью перейти на новый механизм работы с исходниками и синхронизируется с основным репо. Используют его пока только энтузиасты

@nixel2007
Copy link
Member

Да и vb1 будет deprecated однажды

@leemuar
Copy link
Author

leemuar commented Dec 4, 2017

Ну так и я за такой же подход тут агитирую

@theshadowco
Copy link
Collaborator

С VB несколько иная ситуация, все таки не OS пакет.
А с деприкейтом - надо OPM очить работать с зависимостями более интересно + обработка деприкейтов (gentoo+emerge наше все :))

@theshadowco
Copy link
Collaborator

Если сконяемся к отдельному репу, могу форкнуть и оформить, не сложно.

@asosnoviy
Copy link
Member

а я за этот реп, против новой либы. Бампим версию. Обратную совместимость обеспечить не сложно же. + сообщение о депрекейтед.

@EvilBeaver
Copy link
Member

Нене. Давайте в этой репе оставим.

@EvilBeaver
Copy link
Member

Кстати, а что именно "Поломается"?

@theshadowco
Copy link
Collaborator

@EvilBeaver в редми написал - API совсем другой. Но с другой стороны, если менять имя класса (в коде либо в конфиге перекинуть) то будет работать

@asosnoviy
Copy link
Member

Закройте ишуз плз

@nixel2007
Copy link
Member

релиз бы выпустить. @theshadowco не хочешь испробовать opm push?

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

No branches or pull requests

6 participants