On 2013-10-23 04:29, Rosa, Andrea (HP Cloud Services) wrote:
Hi
2. Before submitting the new patch for review it's better to run unit
tests (tox -epy27) and pep8 check (tox -epep8)
Instead of pep8 I think you should run flake8 we moved to that some
months ago[1].
Usually I find always useful to test my changes in devstack.
Regards
--
Andrea Rosa
[1]
http://lists.openstack.org/pipermail/openstack-dev/2013-May/009178.html
tox -epep8 actually runs flake8. It just wasn't renamed to avoid
breaking people's existing workflow.
Also, just running "tox" will run all of the appropriate tests for your
environment, which is probably what you want before pushing changes.
Specifying -e is useful while you're working on something though so you
don't have to run flake8 every time.
-Ben
_______________________________________________
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
