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.

Sign up for GitHub

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

Jump to bottom

Remove kwargs #212

Merged

FoamyGuy merged 1 commit into adafruit:main from FoamyGuy:wrong_args_fix

Nov 15, 2024

Contributor

FoamyGuy commented Nov 14, 2024

Resolves: #202

Remove the kwargs and it's only usage. Incidentally it's only usage will effectively get done automatically now that we aren't using the wildcard.

Confirmed on a device that wrong arguments now raise an exception:

TypeError: unexpected keyword argument 'max_glyphs'
TypeError: unexpected keyword argument 'anchor_position'


          remove kwargs

6379cc7

ladyada approved these changes

View reviewed changes

FoamyGuy merged commit 156c13d into adafruit:main

1 check passed

adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request


          Automated update by Adabot (adafruit/adabot@53e7906)

d768e7b

Updating https://github.com/adafruit/Adafruit_CircuitPython_Display_Shapes to 2.9.0 from 2.8.4:
  > Merge pull request adafruit/Adafruit_CircuitPython_Display_Shapes#75 from FoamyGuy/docs_fix
  > Merge pull request adafruit/Adafruit_CircuitPython_Display_Shapes#74 from FoamyGuy/dynamic_arc

Updating https://github.com/adafruit/Adafruit_CircuitPython_Display_Text to 3.1.7 from 3.1.6:
  > Merge pull request adafruit/Adafruit_CircuitPython_Display_Text#213 from FoamyGuy/bmp_lbl_asc_desc_fix
  > Merge pull request adafruit/Adafruit_CircuitPython_Display_Text#212 from FoamyGuy/wrong_args_fix

Updating https://github.com/adafruit/Adafruit_CircuitPython_DisplayIO_Layout to 2.2.0 from 2.1.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_DisplayIO_Layout#94 from FoamyGuy/linear_layout_new

Updating https://github.com/adafruit/Adafruit_CircuitPython_Bundle/circuitpython_library_list.md to NA from NA:
  > Updated download stats for the libraries

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet