Conversation
|
Лучше строго |
|
И так же там по моему лишние мои коммиты. Там должен быть только один ваш коммит |
4bff5bc to
68f828f
Compare
изме
почему "они" если ты вроде как основной разработчик, нет?) |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
"Они" в данном случае разработчики макса. Я говорил про их протокол и апи |
пон |
|
Тесты не прошли. Исправьте пожалуйста |
|
Проверил. Эти падения не внесены PR #98: целевая ветка В CI PR новый
Добавлять исправления несвязанных фильтров в этот PR не стоит: это расширит его область. Сначала нужно исправить |
c926510 to
d0f9bfd
Compare
Описание
MAX присылает числовой
messageIdв событииNOTIF_MSG_REACTIONS_CHANGED. ПолеReactionUpdateEvent.message_idтеперь имеет типint, поэтому событие проходит валидацию и доходит до обработчика. Регрессионный тест проверяет доставку через dispatcher.Также исправлены причины падения CI в
dev/2.5.0.Filter. Тест проверяет совместную работу функций и составного фильтра, включая остановку приFalse.F.message.linkдоступен для сообщений и вложенных сообщений чата.await.Тип изменений
Связанные задачи / Issue
Fixes #96
Тестирование
uv run pytest --cov=src/pymax --cov-report=term --tb=short -q. Все 165 тестов прошли на Python 3.14.uv run ruff format --check .. Успешно.uv run ruff check .. Успешно.uv run pre-commit run --filesдля изменённых Python-файлов. Оба hook прошли.GitHub Actions в форке на коммите
33080be. Lint и все 165 тестов на Python 3.10, 3.11, 3.12, 3.13 и 3.14 прошли.