https://bugzilla.redhat.com/show_bug.cgi?id=2160509 Carl George 🤠 <carl@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|ERRATA |--- Status|CLOSED |ASSIGNED Keywords| |Reopened --- Comment #8 from Carl George 🤠 <carl@xxxxxxxxxx> --- Optional suggestion, use the PyPI tarball instead of the GitLab one. This is not required by the guidelines, but I believe most Python packagers tend to stick with PyPI tarballs unless it is missing tests or other necessary files. -Source: https://gitlab.com/dslackw/colored/-/archive/%{version}/colored-%{version}.tar.gz +Source: %{pypi_source colored} https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_source_files_from_pypi ================================================================================ There is a build requirement for pytest, but it is not used to run the test, so it should be removed. -# for tests -BuildRequires: python3-pytest ================================================================================ Upstream's TravisCI config claims to run nosetests, but none of the files in the tests directory contain valid unit tests. They instead appear to be scripts to run manually and visually compare the color output. I can see the only thing is %check is an import test. I agree with that approach, but it would also be useful to add a comment about why the upstream tests can't be run. ================================================================================ The LICENSE.txt file is properly picked up by the Python metadata, so listing it explicitly in %files produces a duplicate. The explicit listing should be removed to avoid this. -%license LICENSE.txt -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component https://bugzilla.redhat.com/show_bug.cgi?id=2160509 _______________________________________________ package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
