Skip to content

Conversation

@tconkling
Copy link
Contributor

Drive by cleanup of cli.py:

  • Use f-strings where appropriate
  • Add missing type annotations
  • Remove unnecessary @click.pass_context decorator usage
  • Fix caching imports

* develop: (25 commits)
  Fix CORS acronym in docstring (streamlit#5727)
  Add integration tests for Snowpark (streamlit#5543)
  Release/1.15.0 (streamlit#5720)
  Add audit_frontend_dependencies script to CODEOWNERS (streamlit#5708)
  Add label_visibility option for st.checkbox (streamlit#5705)
  Display existing column names in st.map exception and make st.map work with capital letters (streamlit#5679)
  Plotly Customization (streamlit#5681)
  Turn off theme for now (streamlit#5701)
  Add all vendored code to `make notices` (streamlit#5704)
  Audit frontend licenses (streamlit#5664)
  Surround labels in quotes (streamlit#5703)
  Add info about voting on features (streamlit#5660)
  Update issue labeling scheme to adopt new standards (streamlit#5702)
  Cached media (audio+video) replay (streamlit#5695)
  Fix docstring line wrap (streamlit#5698)
  Use specialized assertion functions (streamlit#5680)
  Release 1.14.1 (streamlit#5693)
  Image replay in cached functions (streamlit#5675)
  Add docstrings for `experimental_allow_widgets` (streamlit#5670)
  Remove `st.write` from `CachedStFunctionWarning` (streamlit#5669)
  ...
@tconkling tconkling merged commit be64d6d into streamlit:develop Nov 18, 2022
@tconkling tconkling deleted the tim/CLICleanup branch November 18, 2022 22:56
tconkling added a commit to tconkling/streamlit that referenced this pull request Nov 18, 2022
* develop:
  cli.py cleanup (streamlit#5737)
  Add CI job to enforce security assessments (streamlit#5709)
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