Spec URL: https://dcavalca.fedorapeople.org/review/python-htmlmin2/python-htmlmin2.spec SRPM URL: https://dcavalca.fedorapeople.org/review/python-htmlmin2/python-htmlmin2-0.1.13-1.fc42.src.rpm Description: This package provides a configurable HTML Minifier with safety features. This is a fork of htmlmin. Fedora Account System Username: dcavalca
This package built on koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=126978023
Copr build: https://copr.fedorainfracloud.org/coprs/build/8406759 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2332972-python-htmlmin2/fedora-rawhide-x86_64/08406759-python-htmlmin2/fedora-review/review.txt Found issues: - python3-pytest7 is deprecated, you must not depend on it. Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/deprecating-packages/ Please know that there can be false-positives. --- This comment was created by the fedora-review-service https://github.com/FrostyX/fedora-review-service If you want to trigger a new Copr build, add a comment containing new Spec and SRPM URLs or [fedora-review-service-build] string.
Hello, I have few suggestion for improving your specfile, just informal, I'm not a maintainer. According to the python guideline:https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/ If is not possible to use the upstream install method:, you should use BuildRequires: python3dist(pytest) Use of multiple licenses should be explicitly defined for all files: https://fedoraproject.org/wiki/Packaging:LicensingGuidelines
I think python3-pytest and python3dist(pytest) are equivalent, it's not a big deal either way. and the file that is under Python 2.0.1 license is specified in the comment so it's fine Package was generated with pyp2spec, simplifying the review. Package was unretired and the diff from the previous spec (FXX) is reasonable, simplifying the review. - package builds and installs without errors on rawhide - test suite is run and all unit tests pass - latest version of the sotware is packaged - license matches upstream specification (BSD-3-Clause AND Python-2.0.1) and is acceptable for Fedora - license file is included with %license in %files - package complies with the Python Packaging Guidelines Package APPROVED. === Recommended post-import python-packagers-sig tasks (use `fedora-sig-onboard onboard pkgname`: - add @python-packagers-sig with "commit" access as package co-maintainer - track package in koschei for all built branches
The Pagure repository was created at https://src.fedoraproject.org/rpms/python-htmlmin2
FEDORA-2024-de7db82619 (python-htmlmin2-0.1.13-1.fc42) has been submitted as an update to Fedora 42. https://bodhi.fedoraproject.org/updates/FEDORA-2024-de7db82619
FEDORA-2024-de7db82619 (python-htmlmin2-0.1.13-1.fc42) has been pushed to the Fedora 42 stable repository. If problem still persists, please make note of it in this bug report.