https://bugzilla.redhat.com/show_bug.cgi?id=2160509
Carl George 🤠 carl@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Resolution|ERRATA |--- Status|CLOSED |ASSIGNED Keywords| |Reopened
--- Comment #8 from Carl George 🤠 carl@redhat.com --- 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/%%7Bversion%7D/colored-%%7Bvers... +Source: %{pypi_source colored}
https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_source_fi...
================================================================================
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