https://bugzilla.redhat.com/show_bug.cgi?id=1289970
--- Comment #10 from Javier Peña <jpena(a)redhat.com> ---
(In reply to Zbigniew Jędrzejewski-Szmek from comment #9)
You don't need to repeat the Summary text. After the first
instance you can
refer the the previous one with %{summary}. And for tests you you can use
"Summary: %{summary} - test files".
Also, "An " in the summary is not useful, listings look better without the
article.
Very nice package.
+ license is acceptable (ASL 2)
+ license file is present, %license is used
- not the latest version (0.7.4 is latest on pypi)
+ no scriptlets
+ %check is present and passes
+ rpmlint is happy (only no-documentation warning)
+ provides and requires seem right
Package is APPROVED.
Please update to the latest version.
About the latest version, it introduces a dependency on openstackdocstheme,
which is not packaged yet. I remember there were some concerns due to its usage
of Google Analytics and the fact it includes files from external references.
Should I go ahead and get a package review for that dependency first?
--
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