search

LyuPo7 / bot

Haskell Echo-bot
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Замечания по коду #4

Closed roman-bodavskiy closed 2 years ago

roman-bodavskiy commented 3 years ago

Замечания hlint не везде исправлены. Например в тех же тестах https://github.com/LyuPo7/bot/blob/master/test/TestBot/Tele/Run.hs

roman-bodavskiy commented 3 years ago

Есть неиспользуемы прагмы, например в модуле Main. Часто используемые можно вынести в package.yaml

roman-bodavskiy commented 3 years ago

config.json лучше вынести в .gitignore чтобы случайно не запушить токен ну и всякие файлы типа data/tele.db

LyuPo7 commented 3 years ago

Исправлено

Замечания hlint не везде исправлены. Например в тех же тестах https://github.com/LyuPo7/bot/blob/master/test/TestBot/Tele/Run.hs

LyuPo7 commented 3 years ago

Исправлено

Есть неиспользуемы прагмы, например в модуле Main. Часто используемые можно вынести в package.yaml

LyuPo7 commented 3 years ago

Добавила

config.json лучше вынести в .gitignore чтобы случайно не запушить токен ну и всякие файлы типа data/tele.db

KateBushueva commented 2 years ago

лучше убрать неиспользуемые расширения как например AllowAmbiguousTypes в модуле Bot.Mode.Mode