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

Chandan Kumar <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected],
                   |                            |[email protected]
              Flags|                            |needinfo?([email protected])



--- Comment #1 from Chandan Kumar <[email protected]> ---
Thanks Jpena for putting the Package Review.

Below is some of my comments:
* The current pypi tarball does not contains docs and tests folder, please use
  github tarball.
Note: I have added a patch upstream to include docs and tests in pypi tarball:
https://github.com/miguelgrinberg/Flask-HTTPAuth/pull/55

* Add -docs and -tests subpackage to the project as well as include %check
section 
  to run unit tests.

Rest looks ok to me.

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