Code review comment for lp:~frankban/juju-quickstart/add-tox
Revision history for this message
| Brad Crittenden (bac) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
bbfa235
(Get the code!)
LGTM. Did not try to QA yet. Tox may be very interesting for lots of our
projects.
https:/
File HACKING.rst (right):
https:/
HACKING.rst:356: parsing ``tox.ini``, dependencies must be listed with
the following rules:
Could you try to re-word that sentence? I'm not sure what you're saying.
https:/
HACKING.rst:358: - each scenario must at least include a
"{[testenv]deps}" line in its deps:
Change the : to ; or ,
https:/
File MANIFEST.in (right):
https:/
MANIFEST.in:24: # tox.ini file, removing it from this list will break
the source distribution
Again, this could be worded better.
https:/
File setup.py (right):
https:/
setup.py:43: """Return Juju Quickstart package data by walking through
the project."""
A better explanation of what "package data" is would be nice.
https:/
setup.py:74: for requirement in deps:
Maybe add
requirement = requirement.strip()
https:/
File tox.ini (right):
https:/
tox.ini:42: basepython = python2.7
Is it required to repeat here?
https:/