MarshalX / yandex-music-api

Неофициальная Python библиотека для работы с API сервиса Яндекс.Музыка
https://yandex-music.rtfd.io
GNU Lesser General Public License v3.0
913 stars 79 forks source link

Пересмотреть документацию, внести правки #285

Open MarshalX opened 4 years ago

MarshalX commented 4 years ago

1. В старых местах документации используется такое:

price (:obj:`yandex_music.Price`): Цена.

Надо обновить до такого:

price (:obj:`yandex_music.Price` | :obj:`None`): Цена.

Обновить только в атрибутах, в аргументах должно быть без этого, но с optional Это касается всех полей, которые являются объектами классов библиотеки, не смотря на опциональность в аргументах.

Опциональность в аргументах влияет на наличие optional.

2. Пересмотреть названия атрибутов и аргументов. Зарезервированные имена допускаются в атрибутах, но не в аргументах.

3. Пересмотреть блоки Raise в докстрингах. Была некорректная ссылка.

4. Очень много TODO у полей, надо разобраться за что они отвечают и задокументировать.

MarshalX commented 4 years ago

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

glebliutsko commented 1 year ago

https://github.com/MarshalX/yandex-music-api/blob/main/yandex_music/album/track_position.py#L14

Не None, а Note

UPD: Решено в #570

MarshalX commented 1 year ago

Ишус чутка устарел, 1 и 2 пункты думаю уже неактуальные

pank-su commented 7 months ago

Указан список идентификаторов треков, по факту только один

https://github.com/MarshalX/yandex-music-api/blob/d844e4770e0263415139c05d2e94ae735b0fca14/yandex_music/client.py#L540