Skip to content

Fix for st.code("") raising an exception in the browser#272

Merged
jrhone merged 1 commit intostreamlit:developfrom
jrhone:fix_st_code_empty
Oct 4, 2019
Merged

Fix for st.code("") raising an exception in the browser#272
jrhone merged 1 commit intostreamlit:developfrom
jrhone:fix_st_code_empty

Conversation

@jrhone
Copy link
Copy Markdown
Contributor

@jrhone jrhone commented Oct 4, 2019

Issue:
#146

Description:

  • Don't pass empty strings to Prism.highlight

@jrhone jrhone requested a review from a team October 4, 2019 12:18
Copy link
Copy Markdown
Contributor

@monchier monchier left a comment

Choose a reason for hiding this comment

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

I would add a test for this.

@jrhone
Copy link
Copy Markdown
Contributor Author

jrhone commented Oct 4, 2019 via email

@jrhone jrhone merged commit a4c9b08 into streamlit:develop Oct 4, 2019
tconkling added a commit to tconkling/streamlit that referenced this pull request Oct 7, 2019
* develop:
  Support cli parameters (streamlit#186)
  Update README.md
  Update README.md (streamlit#282)
  Bug fix: make Streamlit watch files in subfolders again (streamlit#265)
  fix st.code("") (streamlit#272)
  different jslint commands for circleci and not (streamlit#273)
  Batch updates to report elements in the App state (streamlit#194)
  Minor typo in pull_request_template.md (streamlit#262)
  Fix wrong quote used in the provided sample code (streamlit#267)
  Fix websocket port handling (streamlit#263)
  Closing sidebar when clicking outside (streamlit#223)
  Version 0.47.2 (streamlit#212)
  [docs] Changed value in Show progress section of docs/getting_started.md to fix error (Issue streamlit#234) (streamlit#235)
  Update README.md
  Remove checkbox from CLA in PR Template
  Update README.md
  Update README.md
  mapping_demo: catch urllib errors (streamlit#224)
tconkling added a commit to tconkling/streamlit that referenced this pull request Oct 8, 2019
* develop:
  Read config from ${CWD}/.streamlit/config.toml (streamlit#227)
  Fix unsafe_allow_html (streamlit#259)
  Support cli parameters (streamlit#186)
  Update README.md
  Update README.md (streamlit#282)
  Bug fix: make Streamlit watch files in subfolders again (streamlit#265)
  fix st.code("") (streamlit#272)
  different jslint commands for circleci and not (streamlit#273)
  Batch updates to report elements in the App state (streamlit#194)
  Minor typo in pull_request_template.md (streamlit#262)
  Fix wrong quote used in the provided sample code (streamlit#267)
  Fix websocket port handling (streamlit#263)
  Closing sidebar when clicking outside (streamlit#223)
  Version 0.47.2 (streamlit#212)
  [docs] Changed value in Show progress section of docs/getting_started.md to fix error (Issue streamlit#234) (streamlit#235)
  Update README.md
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