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

Add typing and documentation for ScrollingLabel #170

Merged
merged 6 commits into from
May 19, 2022

Conversation

rgrizzell
Copy link
Contributor

I noticed it was missing.

@tekktrik tekktrik requested a review from a team May 12, 2022 17:36
@tekktrik tekktrik added the documentation Improvements or additions to documentation label May 12, 2022
Copy link
Member

@tekktrik tekktrik left a comment

Choose a reason for hiding this comment

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

Thanks for the submission, this is great! I had some feedback, mostly about the use of Optional (and some of these are just thank yous :) ). Let me know if I can help explain anything!

adafruit_display_text/scrolling_label.py Outdated Show resolved Hide resolved
adafruit_display_text/scrolling_label.py Outdated Show resolved Hide resolved
adafruit_display_text/scrolling_label.py Outdated Show resolved Hide resolved
adafruit_display_text/scrolling_label.py Show resolved Hide resolved
adafruit_display_text/scrolling_label.py Outdated Show resolved Hide resolved
adafruit_display_text/scrolling_label.py Outdated Show resolved Hide resolved
adafruit_display_text/scrolling_label.py Outdated Show resolved Hide resolved
adafruit_display_text/scrolling_label.py Outdated Show resolved Hide resolved
adafruit_display_text/scrolling_label.py Outdated Show resolved Hide resolved
docs/api.rst Show resolved Hide resolved
rgrizzell and others added 2 commits May 16, 2022 21:45
Co-authored-by: Alec Delaney <89490472+tekktrik@users.noreply.github.com>
@rgrizzell
Copy link
Contributor Author

Builds are currently failing until an upstream issue can be resolved.
adafruit/Adafruit_Blinka_Displayio#92

Copy link
Member

@tekktrik tekktrik left a comment

Choose a reason for hiding this comment

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

This should be resolved now. Can you pin the minimum version of Adafruit_Blinka_displayio to 0.10.2 in setup.py and requirements.txt so that anyone upgrading can grab the update library? Let me know if you'd like help doing so!

@rgrizzell
Copy link
Contributor Author

I think that'll do it. Documentation builds nicely with the latest version of adafruit-blinka-displayio. Thanks for adding the typing.

Copy link
Member

@tekktrik tekktrik left a comment

Choose a reason for hiding this comment

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

This looks great! Thanks for doing this @rgrizzell!

@tekktrik tekktrik merged commit 88635f3 into adafruit:main May 19, 2022
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request May 20, 2022
Updating https://github.com/adafruit/Adafruit_CircuitPython_BD3491FS to 1.1.9 from 1.1.8:
  > Merge pull request adafruit/Adafruit_CircuitPython_BD3491FS#9 from tekktrik/doc/doc-updates
  > Patch .pre-commit-config.yaml
  > change discord badge
  > Patch: Replaced discord badge image
  > Updated gitignore
  > Update Black to latest.
  > Fixed readthedocs build
  > Consolidate Documentation sections of README

Updating https://github.com/adafruit/Adafruit_CircuitPython_FONA to 2.1.8 from 2.1.7:
  > Merge pull request adafruit/Adafruit_CircuitPython_FONA#19 from FoamyGuy/typing_fallback_py37
  > Patch .pre-commit-config.yaml
  > change discord badge
  > Patch: Replaced discord badge image
  > Update .gitignore

Updating https://github.com/adafruit/Adafruit_CircuitPython_RFM9x to 2.2.5 from 2.2.4:
  > Merge pull request adafruit/Adafruit_CircuitPython_RFM9x#77 from tekktrik/fix/literal-import-failure
  > Patch .pre-commit-config.yaml

Updating https://github.com/adafruit/Adafruit_CircuitPython_VC0706 to 6.0.1 from 6.0.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_VC0706#29 from FoamyGuy/typing_fallback
  > Patch .pre-commit-config.yaml
  > change discord badge
  > Patch: Replaced discord badge image
  > Updated gitignore

Updating https://github.com/adafruit/Adafruit_CircuitPython_Display_Text to 2.22.4 from 2.22.3:
  > Merge pull request adafruit/Adafruit_CircuitPython_Display_Text#170 from rgrizzell/main
  > Patch .pre-commit-config.yaml
  > change discord badge
  > Patch: Replaced discord badge image
  > Updated gitignore

Updating https://github.com/adafruit/Adafruit_CircuitPython_RSA to 1.2.9 from 1.2.8:
  > Merge pull request adafruit/Adafruit_CircuitPython_RSA#30 from FoamyGuy/py37_typing
  > Patch .pre-commit-config.yaml
  > change discord badge
  > Patch: Replaced discord badge image
  > Updated gitignore
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants