https://bugzilla.redhat.com/show_bug.cgi?id=967945 --- Comment #4 from Petr Šabata <psabata@xxxxxxxxxx> --- (In reply to Petr Pisar from comment #1) > TODO: Spell ident as Ident in the summary to be consistent with the > description. > TODO: Spell lookup in the summary the same way as in the description. > > TODO: Correct typo `tree' in the description. > TODO: Quote URL in the description with angle brackets. Thanks. Done. > TODO: Consider unbundling inc::Modules::Install. I prefer using code provided by upstream. > TODO: Constrain `perl(Test::More)' with `>= 0.47'. > TODO: Constrain `perl(POE)' with `>= 0.38'. None of current Fedora or EL releases ships with such ancient versions. That's unnecessary. Anyhow, thank you for the review. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=E19s8UF4hN&a=cc_unsubscribe _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review