Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.debian.maint.python > #17403

review for python-asyncpraw/7.8.1-1

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

hi Aryan,

I took a look at the python-asyncpraw package, up for sponsorship in
the Python team:

* there's a vast diff (doc, tests, tools, ...) between the orig
  tarball uscan pulls in and the one generated from the pristine-tar
  branch.
* copyright: upstream in their README.rst says this is a fork of praw,
  and there's a praw_license.txt file in the root of the sources;
  none of that info shows up in d/copyright though.
* d/salsa-ci.yml present but the CI isn't enabled on the salsa repo.
* why are tests disabled wholesale? Even assuming there were a good
  reason, that should at least be documented - for example by adding a
  comment alongside the setting in d/rules.

I stopped reviewing at this point, as the above might lead to
significant changes.


Once the above comments have been addressed, simply re-add the
package to the IRC channel topic and/or ping me by e-mail.

Back to linux.debian.maint.python | Previous | Next | Find similar

Thread

review for python-asyncpraw/7.8.1-1 Jeroen Ploemen <jcfp@debian.org> - 2026-02-19 14:20 +0100

csiph-web