Skip to content
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

Got unit tests passing again + other cleanup #5

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

victoradan
Copy link
Collaborator

  • Added Makefile to include usual development commands
  • Added tox to run unit tests against Python 2.7
  • Fixed a couple doc issues.

…U and PD commands."

This reverts commit b86000b.

Revert "No, really, this time it's fixed."

This reverts commit a90a8ce.

Revert "Whoops, the bug in pens_updown_to_papr.py wasn't quite fixed. Now it is."

This reverts commit 71bdf46.

Revert "fixed bug in pens_updown_to_papr.py that was overwriting contents of list passed to function."

This reverts commit 35151f9.
- Updated INSTALL
- Added tox for testing against python 2.7
- Added note on compiling hp2xx in Mac OSX
- Fixed tutorial example
- Added exception handling to serial device connection
Fix broken tests, and other improvements
@victoradan victoradan requested a review from drepetto July 7, 2018 20:10
willprice added a commit to willprice/chiplotle that referenced this pull request Aug 20, 2018
See https://github.com/drepetto/chiplotle/blob/13ac28a7645a018a4bc62980cb174627ca6b5b54/chiplotle/tools/hpgltools/pens_updown_to_papr.py

I lifted the code directly from there as the test looked to have been
originally broken, there is an open pull request to fix this issue on
the mainline at drepetto#5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant