-
Notifications
You must be signed in to change notification settings - Fork 10
Fixed Travis Errors and Disabled no-member #12
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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great! Thank you! You did it perfectly. I think however, in the case of these short examples, we're going to do it "incorrectly" to keep the examples cleaner. In 99% of cases, this is exactly how I would want it done, but there's already so little to these examples that doing it properly doesn't make the most sense for readability.
examples/slideshow_simpletest.py
Outdated
|
||
# Create the slideshow object that plays through once alphabetically. | ||
#pylint: disable=no-member |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please place this below the imports.
examples/slideshow_simpletest.py
Outdated
slideshow = SlideShow(board.DISPLAY, pulseio.PWMOut(board.TFT_BACKLIGHT), folder="/", | ||
loop=False, order=PlayBackOrder.ALPHABETICAL) | ||
#pylint: enable=no-member |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this.
examples/slideshow_touch.py
Outdated
|
||
forward_button = touchio.TouchIn(board.TOUCH4) | ||
back_button = touchio.TouchIn(board.TOUCH1) | ||
|
||
brightness_up = touchio.TouchIn(board.TOUCH3) | ||
brightness_down = touchio.TouchIn(board.TOUCH2) | ||
|
||
#pylint: disable=no-member |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above.
examples/slideshow_touch.py
Outdated
slideshow = SlideShow(board.DISPLAY, pulseio.PWMOut(board.TFT_BACKLIGHT), folder="/", | ||
auto_advance=False, dwell=0) | ||
#pylint: enable=no-member |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apologies for not being clearer! Please leave the # pylint: disable=no-member
in each file, but place it below the imports. Do not include the associated enable
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect, thank you!
Updating https://github.com/adafruit/Adafruit_CircuitPython_ESP32SPI to 1.1.1 from 1.1.0: > Merge pull request adafruit/Adafruit_CircuitPython_ESP32SPI#7 from brentru/expose-requests-more Updating https://github.com/adafruit/Adafruit_CircuitPython_Slideshow to 1.0.3 from 1.0.2: > Merge pull request adafruit/Adafruit_CircuitPython_Slideshow#12 from makermelissa/master > Merge pull request adafruit/Adafruit_CircuitPython_Slideshow#10 from tannewt/update_for_4 Updating https://github.com/adafruit/Adafruit_CircuitPython_Bundle/circuitpython_library_list.md to NA from NA: > Added the following libraries: Adafruit_CircuitPython_Bitmap_Font, Adafruit_CircuitPython_Display_Text
Fixes #11.