Skip to content

Fix/selectbox typings#4194

Merged
vdonato merged 2 commits intostreamlit:developfrom
whitphx:fix/selectbox-typings
Dec 21, 2021
Merged

Fix/selectbox typings#4194
vdonato merged 2 commits intostreamlit:developfrom
whitphx:fix/selectbox-typings

Conversation

@whitphx
Copy link
Copy Markdown
Contributor

@whitphx whitphx commented Dec 17, 2021

📚 Context

Please describe the project or issue background here

  • What kind of change does this PR introduce?

    • Bugfix
    • Feature
    • Refactoring
    • Other, please describe:

🧠 Description of Changes

The return type of format_func should not be restricted to str as it will be casted to str afterwards.
Moreover, the type of the raw label which is also the input of format_func is Any, so the return type of the func should also be Any.

  • Add bullet points summarizing your changes here

    • This is a breaking API change
    • This is a visible (user-facing) change

Revised:

Insert screenshot of your updated UI/code here

Current:

Insert screenshot of existing UI/code here

🧪 Testing Done

  • Screenshots included
  • Added/Updated unit tests
  • Added/Updated e2e tests

🌐 References

Does this depend on other work, documents, or tickets?

No


Contribution License Agreement

By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

Copy link
Copy Markdown
Collaborator

@vdonato vdonato left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution @whitphx, this LGTM!

@vdonato vdonato merged commit 06330bd into streamlit:develop Dec 21, 2021
@whitphx whitphx deleted the fix/selectbox-typings branch December 21, 2021 06:29
tconkling added a commit to tconkling/streamlit that referenced this pull request Jan 3, 2022
* develop:
  Fix hello demo type annotation (streamlit#4228)
  Release 1.3.1 (streamlit#4220)
  Improve beta_ deprecation message (streamlit#4219)
  Changing Image Algorithm to Bilinear (streamlit#4159)
  Allow columns to be rendered to create spacing (streamlit#4217)
  Fix issue with hidden balloons (approach 2) (streamlit#4204)
  Fix mypy errors from 0.930 release (streamlit#4218)
  Fix/selectbox typings (streamlit#4194)
  Support running tests locally on Apple Silicon (streamlit#4185)
  Stop screencast recorder when user removes permission / stops using browser button. (streamlit#4180)
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