https://bugzilla.redhat.com/show_bug.cgi?id=866982 Vít Ondruch <vondruch@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? | Flags| |fedora-review+ --- Comment #9 from Vít Ondruch <vondruch@xxxxxxxxxx> --- * pushd paired with popd - This is minor nit and has no influence on anything, but these are pair commands and they should be used in pair IMO (see %check section). BTW Upstream question: What is the purpose of two identical READMEs? Otherwise the package looks good => APPROVED. Since you proven that you understand the guidelines by passing two packages through review and poking around for some informal reviews, I am going to sponsor you as well. Enjoy your new privileges for good of community ;) -- You are receiving this mail because: You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review