New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecate openag_python, move remaining functionality to openag_brain/.../openag_lib module. #275

Merged
merged 8 commits into from Jun 13, 2017

Conversation

Projects
None yet
2 participants
@rbaynes
Member

rbaynes commented Jun 13, 2017

When reviewing, please run the tests!
py.test ~/catkin_ws/src/openag_brain/test/test_lib/

@rbaynes rbaynes requested review from sp4ghet and gordonbrander Jun 13, 2017

@rbaynes rbaynes self-assigned this Jun 13, 2017

@@ -47,6 +47,9 @@ sudo apt-get install -y -q couchdb python-setuptools
sudo easy_install pip
sudo pip install rosdep rosinstall_generator wstool rosinstall
# Install some python dependencies that there aren't ros packages for.
sudo pip install voluptuous HTTPretty pytest

This comment has been minimized.

@sp4ghet

sp4ghet Jun 13, 2017

Collaborator

Do we use HTTPretty for testing or something?

This comment has been minimized.

@rbaynes

rbaynes Jun 13, 2017

Member

@spaghet yes, in the tests, along with pytest. It is a cool module.

@sp4ghet

Looks good to me

woops

@rbaynes rbaynes merged commit 81b16a5 into develop Jun 13, 2017

@sp4ghet sp4ghet deleted the deprecate_openag_python_rob branch Jun 15, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment