https://bugzilla.redhat.com/show_bug.cgi?id=1385105 Chandan Kumar <chkumar@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |chkumar@xxxxxxxxxx --- Comment #3 from Chandan Kumar <chkumar@xxxxxxxxxx> --- Hello David, Thanks for submitting the review. Below is the few comments: [1.] Please also include python-six, python-sseclient (Which is not packaged yet in fedora) as a Requires as they are required by python-marathon. For more info check this: https://paste.fedoraproject.org/453831/70149814/ [2.] for LICENSE and README.md file Sources, please submit a patch upstream and put the link in the spec file. Rest looks ok to me on a first glance. Thanks, Chandan Kumar -- 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 To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx