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

Chandan Kumar <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]



--- Comment #3 from Chandan Kumar <[email protected]> ---
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 -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to