https://bugzilla.redhat.com/show_bug.cgi?id=1156619 --- Comment #6 from Parag AN(पराग) <panemade@xxxxxxxxx> --- This looks good now :) Review: + is OK - is Needs Work + Package built successful in mock (f22 x86_64) - rpmlint on generated rpms gave output python-tzlocal.src: W: spelling-error Summary(en_US) tzinfo -> tinfoil python-tzlocal.src: W: spelling-error %description -l en_US tzinfo -> tinfoil python3-tzlocal.noarch: W: spelling-error Summary(en_US) tzinfo -> tinfoil python3-tzlocal.noarch: W: summary-not-capitalized C tzinfo object for the local timezone python3-tzlocal.noarch: W: spelling-error %description -l en_US tzinfo -> tinfoil python-tzlocal.noarch: W: spelling-error Summary(en_US) tzinfo -> tinfoil python-tzlocal.noarch: W: spelling-error %description -l en_US tzinfo -> tinfoil 3 packages and 0 specfiles checked; 0 errors, 7 warnings. + Source verified with upstream as sha256sum srpm tarball: 4d9ddb8d5eab086e3a7c504c6e994ffa85df43e40da4d6be776218be051c677a upstream tarball: 4d9ddb8d5eab086e3a7c504c6e994ffa85df43e40da4d6be776218be051c677a + License is CCO and included in LICENSE.txt file + follow python packaging guidelines + rest also looks as per packaging guidelines. Suggestion: 1) Summary for main package python-tzlocal should be "A Python module that tries to figure out what your local timezone is". This is a nice summary as can be seen on github page 2) %description should contain some more information like This Python module returns a tzinfo object with the local timezone information. It requires pytz, and returns pytz tzinfo objects. This module attempts to fix a glaring hole in pytz, that there is no way to get the local timezone information, unless you know the zoneinfo name. 3) use above for python3-tzlocal %description Fix above and for this if you want you don't need to bump release number and package will be ready to be approved. -- 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