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

Changed toggle() to value() #280

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

FunDeckHermit
Copy link

The function toggle() is not universal, the ESP32 cannot use it.

(Hi! 👋 Thanks for sending a pull request! Please make sure you click the link above to view the contribution guidelines, then fill out the blanks below.)

What does this implement/fix? Explain your changes.

toggle() is only available on the "mimxrt, samd, rp2 ports."

Does this close any currently open issues?

No

Any relevant logs, error output, etc?

(If it’s long, please paste to https://gist.github.com and insert the link here)
No

Any other comments?

Flashing MicroPython using webserial is very easy.
You might want to add a few lines about webserial in the requirements.

Where has this been tested?

Operating system:
Windows 11 24H2

VSCode version:
V1.96.4

The function toggle() is not universal, the ESP32 cannot use it.
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.

1 participant