Skip to content
This repository was archived by the owner on Dec 23, 2021. It is now read-only.

changed package import name from cpx to cp - #173

Merged
Andrea Mah (andreamah) merged 2 commits into
devfrom
users/t-anmah/cpx-library-name-change
Jan 22, 2020
Merged

Andrea Mah (andreamah) merged 2 commits into
devfrom
users/t-anmah/cpx-library-name-change

Conversation

@andreamah

Copy link
Copy Markdown
Contributor

Description:

Allowed for both imports as:
from adafruit_circuitplayground import cp
and
from adafruit_circuitplayground.express import cpx

This was requested since the first import statement above is how most tutorials present the import statement.

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

Limitations:

Testing:

  • Test with simulation and deploy to board

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules

@andreamah

Copy link
Copy Markdown
Contributor Author

It seems that CircuitPython's standard import statement changed a few months ago. It now can automatically detect the CP board type (Bluefruit vs Express). See: adafruit/Adafruit_CircuitPython_CircuitPlayground@cfa0053#diff-83e362785431178a0efc45ce23e99323

@vandyliu

Copy link
Copy Markdown
Contributor

Nice fix and nice find!

@isadorasophia Isa (isadorasophia) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

That was simple enough 😮 Cool!

Comment thread src/adafruit_circuitplayground/__init__.py
@vandyliu
Vandy Liu (vandyliu) deleted the users/t-anmah/cpx-library-name-change branch January 23, 2020 03:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants