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

Remove unneeded parameter #26

Merged
merged 1 commit into from
Oct 1, 2024
Merged

Remove unneeded parameter #26

merged 1 commit into from
Oct 1, 2024

Conversation

jepler
Copy link
Member

@jepler jepler commented Sep 30, 2024

black_bits_inverted defaults to False in the core, so this is redundant.

By removing this line, an override value can be provided via kwargs if the default is incorrect.

For a display where the default value gives white-on-black, such as https://www.adafruit.com/product/4195, passing in black_bits_inverted=True to the constructor also provides an easy way to invert the whole display, giving white-on-black.

`black_bits_inverted` defaults to False in the core, so this is
redundant.

By removing this line, an override value can be provided via kwargs
if the default is incorrect.
Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

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

Thank you!

@tannewt tannewt merged commit 2cc78cd into main Oct 1, 2024
2 checks passed
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Oct 3, 2024
Updating https://github.com/adafruit/Adafruit_CircuitPython_PM25 to 2.1.17 from 2.1.16:
  > Merge pull request adafruit/Adafruit_CircuitPython_PM25#28 from snkYmkrct/main

Updating https://github.com/adafruit/Adafruit_CircuitPython_SCD4X to 1.4.4 from 1.4.3:
  > Merge pull request adafruit/Adafruit_CircuitPython_SCD4X#23 from snkYmkrct/main

Updating https://github.com/adafruit/Adafruit_CircuitPython_SHT4x to 1.0.20 from 1.0.19:
  > Merge pull request adafruit/Adafruit_CircuitPython_SHT4x#15 from snkYmkrct/main

Updating https://github.com/adafruit/Adafruit_CircuitPython_SSD1680 to 1.2.3 from 1.2.2:
  > Merge pull request adafruit/Adafruit_CircuitPython_SSD1680#26 from adafruit/redundant-constructor

Updating https://github.com/adafruit/Adafruit_CircuitPython_Logging to 5.5.1 from 5.5.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_Logging#65 from PCPartPicker/main

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants