https://bugzilla.redhat.com/show_bug.cgi?id=1869719



--- Comment #2 from Jaroslav Prokop <jar.pro...@volny.cz> ---
Looks fine, I just have 2 nits:

There is some documentation in `doc` directory in the image_processing project
that I think should be included since it looks like it has more content than
the generated docs.
Maybe transform it into some kind of manpage or include it in the *-doc
subpackage?

And just a triviality:
This `%{gem_instdir}/image_processing.gemspec` could use a macro so it becomes
`%{gem_instdir}/%{gem_name}.gemspec`.


-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org

Reply via email to