Skip to content

Remove cp 6 fallback #29

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 2 commits into from
Feb 19, 2022
Merged

Conversation

FoamyGuy
Copy link
Contributor

This changes removes all pixel_shader CP 6 compatibility code. So moving forward the library is compatible with CP 7+ only.

I also removed the now-artificial limit on number of things in the addon_group.

And changed the remaining instances of files being opened before passed to OnDiskBitmap() constructor to instead pass the filenames and let the file be managed internally.

Tested successfully with modified version of turtle_bgpic_changeturtle.py script on PyPortal 7.2.0 rc0 as well as a few of the other fractal examples.

Copy link
Contributor

@dhalbert dhalbert left a comment

Choose a reason for hiding this comment

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

Great! Thank you!

@dhalbert dhalbert merged commit 5a5ef0e into adafruit:main Feb 19, 2022
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Feb 23, 2022
Updating https://github.com/adafruit/Adafruit_CircuitPython_ESP32SPI to 4.0.2 from 4.0.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_ESP32SPI#157 from Neradoc/patch-1
  > Fixed readthedocs build
  > Consolidate Documentation sections of README

Updating https://github.com/adafruit/Adafruit_CircuitPython_INA260 to 1.3.4 from 1.3.3:
  > Merge pull request adafruit/Adafruit_CircuitPython_INA260#19 from gpongelli/patch-2
  > Fixed readthedocs build
  > Consolidate Documentation sections of README

Updating https://github.com/adafruit/Adafruit_CircuitPython_LSM6DS to 4.3.0 from 4.2.3:
  > Merge pull request adafruit/Adafruit_CircuitPython_LSM6DS#49 from myllyja/add-lsm6ds3
  > Fixed readthedocs build
  > Consolidate Documentation sections of README

Updating https://github.com/adafruit/Adafruit_CircuitPython_BusDevice to 5.1.8 from 5.1.7:
  > Merge pull request adafruit/Adafruit_CircuitPython_BusDevice#86 from tekktrik/hotfix/require-python-version-pt2

Updating https://github.com/adafruit/Adafruit_CircuitPython_MiniMQTT to 5.2.2 from 5.2.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_MiniMQTT#103 from tekktrik/doc/minor-updates
  > Fixed readthedocs build
  > Consolidate Documentation sections of README

Updating https://github.com/adafruit/Adafruit_CircuitPython_turtle to 2.2.5 from 2.2.4:
  > Merge pull request adafruit/Adafruit_CircuitPython_turtle#29 from FoamyGuy/remove_cp_6_fallback
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