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

[СТАТУС ОБСЛУЖИВАНИЯ] #123

Closed
YariKartoshe4ka opened this issue May 30, 2022 · 7 comments
Closed

[СТАТУС ОБСЛУЖИВАНИЯ] #123

YariKartoshe4ka opened this issue May 30, 2022 · 7 comments

Comments

@YariKartoshe4ka
Copy link
Collaborator

YariKartoshe4ka commented May 30, 2022

Последний коммит был совершен 4 года назад, а последний релиз был выпущен 7 лет назад. За все это время накопилось множество Issue и PR, на которые автор не отвечает. Судя по этим фактам, можно сделать только один вывод - к сожалению, данная библиотека больше не обслуживается. Однако она до сих пор пользуется популярностью (~19k скачиваний в месяц), и поэтому я решил создать дружелюбный форк (репо), чтобы продолжить обслуживание данной библиотеки.

Пока что была добавлена поддержка Python 3.10 и обновлена оболочка проекта
PR приветствуется больше всего

Форк: https://github.com/vk-paw/vk-paw
Неаткульно: библиотека продолжает обслуживаться в данном репозитории (мною)

@f213
Copy link

f213 commented May 30, 2022

PR приветствуется больше всего

Скажи, а какие проблемы, кроме несовместимости с современным питоном, ты видишь? Чем помочь?

@YariKartoshe4ka
Copy link
Collaborator Author

Скажи, а какие проблемы, кроме несовместимости с современным питоном, ты видишь? Чем помочь?

Я вижу несколько проблем:

  • UserAPI давно просрочился и с его помощью нельзя получить токен
  • После рефакторинга в vk.session часть документации устарела. Документацию следовало бы расширить, как минимум написать про обработку (перехват) таких насущных ошибок как Captcha needed и Too many requests per second (основываясь на своем опыте). Также я считаю, что доки стоит перевести на русский, ведь пользователи данной библиотеки (да и всего ВК в целом) преимущественно русскоговорящие.
  • Покрытие тестами в данном репо составляет лишь 61%, а в форке и то 49% (часть тестов, требующих доступ к VK API, отключена), что не может стать убедительным аргументом работоспособности и надежности библиотеки

Ну и просто нужно взять во внимание открытые Issue и PR в этом репо

@f213
Copy link

f213 commented May 31, 2022

UserAPI давно просрочился и с его помощью нельзя получить токен

Я записал себе в беклог эту задачу. Не буду ничего обещать, но думаю, найду несколько часов на следующей неделе.

@voronind
Copy link
Owner

voronind commented Jun 1, 2022

Привет. У меня нет времени на обслуживание этого модуля. Готов дать доступ на редактирование.
Что касается PyPI. Надо будет настроить gh actions (предпочтительнее) или создать токен PyPI для обновления

@YariKartoshe4ka
Copy link
Collaborator Author

Готов дать доступ на редактирование

Да было бы круто. Я перенесу сделанные в форке коммиты.

Что касается PyPI. Надо будет настроить gh actions (предпочтительнее) или создать токен PyPI для обновления

Я уже сделал workflow для авторелиза на PyPI по тегам. От тебя нужен токен для загрузки новых версий, можешь сохранить в secrets под именем PYPI_API_TOKEN

@YariKartoshe4ka
Copy link
Collaborator Author

@voronind добавь в кураторы на Read the Docs, хочу попозже обновить документацию, а она фэйлится. Ник такой же: YariKartoshe4ka

@voronind
Copy link
Owner

voronind commented Jun 5, 2022 via email

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

3 participants