https://bugzilla.redhat.com/show_bug.cgi?id=1418428 Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |zbyszek@xxxxxxxxx Assignee|nobody@xxxxxxxxxxxxxxxxx |zbyszek@xxxxxxxxx Flags| |fedora-review+ --- Comment #1 from Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> --- Source0: https://pypi.python.org/packages/a8/5b/35f5e9c23b533a676b831c695671b54d2d0781d121989ee07bf80c290f78/retryz-0.1.8.zip → use %{version} %global summary Retry decorator with a bunch of configuration parameters Summary: %{summary} → just put the text in Summary:, this will also define the macro by itself. You can use it below. %description should end in a dot. py.test-2.7 → py.test-2, not that it matters too much. + package name is OK + license is acceptable (ASL 2.0) + license is specified correctly + modern python packaging template is used + P/R/BR look correct + %python_provide are present + builds and installs OK - fedora-review flags an issue: dist tag is not present. See https://fedoraproject.org/wiki/Packaging:DistTag. Package is APPROVED. Please fix the %description and Release fields when uploading, and consider the other changes suggested above. -- 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 _______________________________________________ package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx