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

Chandan Kumar <chku...@redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |chku...@redhat.com



--- Comment #3 from Chandan Kumar <chku...@redhat.com> ---
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

Reply via email to