Добавление интеграции с Matrix

[asmfreak]

Данный запрос на слияние добавляет интеграцию с распределённой сетью чатов Matrix.

Помимо этого:

1. сборочные скрипты для документации были обновлены до самой свежей версии (mkdocs-material 5.x).
2. в документации исправлены внутренние ссылки - вместо страницы /emupdate, например, ссылка со страницы confetti вела на страницу /confetti/emupdate, что неправильно.

[mdevaev]

Привет. Спасибо за патч. Я написал немного замечаний, прогоните еще make tox, пожалуйста, чтобы код соответствовал правилам линтеров.

После фиксов я смержу и поправлю остальное сам в течение недели, я немного занят в ближайшее время.

PS: Вопрос не по теме. Вы случайно не пользуетесь нонейм-клабом?

[asmfreak]

Привет. Простите, но я замечаний не вижу? Над линтерами поработаю, вчера не было сил их допилить.

Мне для работы интеграции пришлость притащить модуль matrix-nio. Я думаю добавить дополнительные цели в setup.py следующим образом:

setup(
    # ...
    extras_require={
        'matrix':  ['matrix-nio']
    },
    # ...
)

Нормально ли это с вашей точки зрения?

PS: Нет. Не пользуюсь

[mdevaev]

Так вот же выше они все перечислены.

Можно добавить не в опциональные, а в основные. В PKGBUILD еще допишите python-matrix-nio

[asmfreak]

Я сомневаюсь в целесообразности добавления в основные - всё же matrix-nio тянет за собой очень много дополнительных сторонних пакетов, которые не нужны другим пользователям emonoda, кроме тех, кому явно нужна поддержка Matrix.

[mdevaev]

Окей по поводу зависимостей. Тогда вынесете импорт наверх, как я в ревью указал, и напишите что-то типа такой логики: https://github.com/mdevaev/emonoda/blob/master/emonoda/plugins/clients/ktorrent.py#L37 https://github.com/mdevaev/emonoda/blob/master/emonoda/plugins/clients/ktorrent.py#L50

[asmfreak]

Вроде бы всё исправил.

[mdevaev]

Окей, спасибо. Смержу и дофикшу сам, когда руки дойдут. В течение недели, я думаю.

[vitlav]

Окей, спасибо. Смержу и дофикшу сам, когда руки дойдут. В течение недели, я думаю. И тут проходит почти год и...

[mdevaev]

Времени вообще нет, стартап же с КВМ.

[asmfreak]

Я бы хотел разделить этот PR на несколько небольших, которые проще было бы слить. Давайте закроем этот пока что?

[mdevaev]

Окей