-
Notifications
You must be signed in to change notification settings - Fork 31
Added example library #9
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
Conversation
thanks! looks like there's a few pylint issues -
|
Not sure why the build is failing. It's been tested and works in the real world. All previous Travis complaints (all cosmetic) have been addressed |
ahhh the linter is picky - @kattni can take a look tomorrow, looks like the two warnings are minor! |
@zlite pylint errors here: https://travis-ci.com/adafruit/Adafruit_CircuitPython_MotorKit/builds/94135005 |
Yeah, those linter complaints are silly: C: 71, 0: Line too long (107/100) (line-too-long) |
You can just make line 52 be a |
I guess I could, but it's perfectly fine as it is, too. I think our linter friend is wrong in both these cases ;-) I'm going to leave it here. If you like it, pull it! If not, no hard feelings |
no biggie - thanks for the PR |
thanks @kattni !! |
Updating https://github.com/adafruit/Adafruit_CircuitPython_PN532 to 2.0.1 from 2.0.0: > Merge pull request adafruit/Adafruit_CircuitPython_PN532#9 from adafruit/setup-py Updating https://github.com/adafruit/Adafruit_CircuitPython_MotorKit to 1.2.2 from 1.2.1: > Merge pull request adafruit/Adafruit_CircuitPython_MotorKit#9 from zlite/master
Similar to Robot.py in previous Python Adafruit Motor Hat Library.
Also added a robot.steer function. Use form: robot.steer(speed, direction) where both speed and direction are +/- 0-1