Size: a a a

2020 April 06

МБ

Максим Барулин in pro.elixir
Ilya Borovitinov
А ещё у вас весьма неочевидное расположение файлов - захотел посмотреть, что же находится за use BotEx.ModuleHandler, пришлось все папки перерыть, где этот файл находится
эм, BotEx.ModuleHandler - макрос, лежит в папке макросов
источник

IB

Ilya Borovitinov in pro.elixir
Ну если придираться, то `BotEx.ModuleHandler - модуль, и то, что он умеет в `__using__ -  не значит, что это его единственное использование
источник

IB

Ilya Borovitinov in pro.elixir
По крайней мере мне это не очень очевидно
источник

LL

Lama Lover in pro.elixir
Максим Барулин
эм, BotEx.ModuleHandler - макрос, лежит в папке макросов
В эликсир принято такое раположение файлов:
Если в файле описан модуль MyProject.MyModule.MySubmodule, то лежать он будет как lib/my_project/my_module/my_submodule.ex
источник

IB

Ilya Borovitinov in pro.elixir
Lama Lover
В эликсир принято такое раположение файлов:
Если в файле описан модуль MyProject.MyModule.MySubmodule, то лежать он будет как lib/my_project/my_module/my_submodule.ex
Вот, да, я об этом
источник

LL

Lama Lover in pro.elixir
В большинстве структур неправильно описаны типы
источник

IB

Ilya Borovitinov in pro.elixir
Максим Барулин
эм, BotEx.ModuleHandler - макрос, лежит в папке макросов
И даже там он лежит в файле, которое не соотевтсвует названию модуля
источник

LL

Lama Lover in pro.elixir
@Slavenin я позже прочитаю код полностью и сделаю ревью
источник

МБ

Максим Барулин in pro.elixir
ок, благодарю, только немного подробнее, хотя бы с одним примером как надо
источник

IB

Ilya Borovitinov in pro.elixir
А ещё про BotEx.Modulehandler - у вас он реализует GenServer-овые коллбеки handle_cast, однако зачем-то требует отдельной вставки use GenServer
источник

IB

Ilya Borovitinov in pro.elixir
может быть можно было добавить эту строку в __using__?
источник

IB

Ilya Borovitinov in pro.elixir
Ещё непонятно использование второго макроса, BotEx.Modulehandler.Init. Какой кейс использования его отдельно от основного макроса?
источник

МБ

Максим Барулин in pro.elixir
Ilya Borovitinov
Ещё непонятно использование второго макроса, BotEx.Modulehandler.Init. Какой кейс использования его отдельно от основного макроса?
можно сделать инициализацию модуля, если она должна отличаться от дефолтной и пользоваться состоянием
источник

IB

Ilya Borovitinov in pro.elixir
А ещё get_cmd_name у вас не описан ни в каком behaviour, хотя вы требуете его реализации от хендлера. Тогда непонятно предназначение BotEx.Behaviours.Handler
источник

МБ

Максим Барулин in pro.elixir
Ilya Borovitinov
А ещё get_cmd_name у вас не описан ни в каком behaviour, хотя вы требуете его реализации от хендлера. Тогда непонятно предназначение BotEx.Behaviours.Handler
согласен, упустили
источник

LL

Lama Lover in pro.elixir
Максим Барулин
согласен, упустили
А ты пускал dialyzer, credo ?
источник

МБ

Максим Барулин in pro.elixir
Диалайзер, который в вскоде работает
источник

МБ

Максим Барулин in pro.elixir
Он выдаёт странную ошибку про под типы и супер типы, но я так и не нагуглил что там к чему поправляется
источник

МБ

Максим Барулин in pro.elixir
И как один и тот же тип может отличаться сам от себя
источник

f

filin49 in pro.elixir
Максим Барулин
Всем привет. Сегодня выложили в опен сорс наш микро фреймворк для ботов с либой под телегу https://github.com/bot-ex/botex хотелось бы услышать ваше мнение о нашем подходе 😃
О, для телеги - эт хорошо 👍🏼
источник