https://bugzilla.redhat.com/show_bug.cgi?id=1234208 Pete Travis <me@xxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks|177841 (FE-NEEDSPONSOR) | Flags| |fedora-review? --- Comment #16 from Pete Travis <me@xxxxxxxxxxxxxx> --- Hey Major, a few observations: - initial package release is 3? - missing %license - There are no docs. Maybe put Scripts in %doc as examples ? - %doc README.md CHANGELOG - "Pure-Python" reads like a Specific Thing with a formal definition; say 'pure Python' - Where does the BuildRequires on python?-nose come from? The `unittest` module in Tests/tests/py is provided by python?-libs. - Strip shebang fom pagerange.py, keep rpmlint happy. - SRPMs for reviews should be built against rawhide. Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review