Skip to content

Convert text object to string. #31

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 1 commit into from
Feb 2, 2020
Merged

Conversation

kattni
Copy link
Contributor

@kattni kattni commented Feb 1, 2020

Allows for providing float object, such as acceleration or gyro data, without needing to convert to string in user code.

@kattni kattni requested a review from a team February 1, 2020 20:57
@jepler
Copy link
Contributor

jepler commented Feb 2, 2020

In chat with kattni, I mentioned that maybe this would affect how bytes objects were displayed. I was wrong about that (as of now, it's an error to pass a bytes object at all), but it does affect how lists and tuples of strings would be displayed.

Before this PR, l.text = ['a', 'b', 'c'] would behave approximately as l.text = 'abc', but after it would show as the bracketed list. Similar for tuples.

I don't think this necessarily means this PR shouldn't be merged, because to me it is very defensible that you could use "anything print()able" with a Label, even if the property is called "text".

@ladyada
Copy link
Member

ladyada commented Feb 2, 2020

yah folks should format() to hearts desire if they dont want str()

@ladyada ladyada merged commit 36e199a into adafruit:master Feb 2, 2020
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Feb 3, 2020
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.

3 participants