Skip to content

Conversation

@FoamyGuy
Copy link
Collaborator

These changes update all projects that use OnDiskBitmap to use the new pixel_shader method from adafruit/circuitpython#4823

These changes are necessary in order to run the projects on CircuitPython 7.0.0.beta-3 and newer.

Once support for CircuitPython 6.x is dropped these could be amended further to remove the getattr call and instead just directly pass bitmap.pixel_shader In the meantime the changes in this PR allow the projects to work on both older versions and newer versions of CircuitPython

Copy link
Contributor

@jepler jepler left a comment

Choose a reason for hiding this comment

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

I skimmed and didn't see any problems with the changes. However, there are merge conflicts which will need to be addressed, I think because I also merged some PRs to fix the max group size. This may indicate needing to work more closely with @lesamouraipourpre when both kinds of updates affect a single file. In any case, thank you!

@FoamyGuy
Copy link
Collaborator Author

I will resolve the merge conflicts tonight to get this back to passing.

# Conflicts:
#	Buckaroo_Plant_Care_Bot/buckaroo_plant_care_bot.py
Copy link
Contributor

@jepler jepler 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!

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