Skip to content

Revert "Remove call to _update_coils in __init__" #30

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

Merged
merged 1 commit into from
May 31, 2019

Conversation

tannewt
Copy link
Member

@tannewt tannewt commented May 31, 2019

Reverts #29

This isn't appropriate because the library isn't designed for use when the pins are connected to something else. Starting with coils energized is a perfectly valid start state and changing it may break uses of the library. More details here: adafruit/Adafruit_CircuitPython_Crickit#17 (comment)

@tannewt tannewt requested review from ladyada and makermelissa May 31, 2019 18:37
@ladyada
Copy link
Member

ladyada commented May 31, 2019

ok!

Copy link
Collaborator

@makermelissa makermelissa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good.

@tannewt tannewt merged commit ddc7484 into master May 31, 2019
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Jun 1, 2019
Updating https://github.com/adafruit/Adafruit_CircuitPython_CharLCD to 3.2.1 from 3.2.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_CharLCD#39 from caternuson/iss36

Updating https://github.com/adafruit/Adafruit_CircuitPython_ST7735 to 1.0.2 from 1.0.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_ST7735#5 from makermelissa/master

Updating https://github.com/adafruit/Adafruit_CircuitPython_ST7735R to 1.0.2 from 1.0.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_ST7735R#6 from makermelissa/master

Updating https://github.com/adafruit/Adafruit_CircuitPython_AzureIoT to 1.0.2 from 1.0.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_AzureIoT#3 from brentru/patch-return-message

Updating https://github.com/adafruit/Adafruit_CircuitPython_ImageLoad to 0.9.1 from 0.8.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_ImageLoad#14 from tannewt/move_tests
  > Merge pull request adafruit/Adafruit_CircuitPython_ImageLoad#12 from crookedstorm/feature-tests
  > Merge pull request adafruit/Adafruit_CircuitPython_ImageLoad#13 from matt-land/bugfix-fix-the-build-sphinx
  > Merge pull request adafruit/Adafruit_CircuitPython_ImageLoad#7 from tannewt/use_bitmap_subscr
  > Merge pull request adafruit/Adafruit_CircuitPython_ImageLoad#11 from crookedstorm/docs-add-development-guide

Updating https://github.com/adafruit/Adafruit_CircuitPython_Motor to 2.0.0 from 1.5.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_Motor#30 from adafruit/revert-29-master
@evaherrada evaherrada deleted the revert-29-master branch June 21, 2021 13:58
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.

3 participants