Skip to content

Conversation

@mayagbarnes
Copy link
Collaborator

Describe your changes

Add the target="_blank" attribute for README links so they open in new tab

Testing Plan

  • No tests needed -> changes isolated to README

@mayagbarnes mayagbarnes added change:chore PR contains maintenance or housekeeping change security-assessment-completed Security assessment has been completed for PR impact:internal PR changes only affect internal code labels Jun 5, 2023
@mayagbarnes mayagbarnes marked this pull request as ready for review June 5, 2023 22:41
@snehankekre snehankekre self-requested a review June 6, 2023 04:52
@mayagbarnes mayagbarnes merged commit ff56b21 into develop Jun 6, 2023
@mayagbarnes mayagbarnes deleted the readme branch June 6, 2023 16:42
tconkling added a commit to tconkling/streamlit that referenced this pull request Jun 6, 2023
* develop:
  Remove tensorflow and pytorch from test requirements (streamlit#6807)
  Fix readme links (streamlit#6800)
  Update PR template to be simpler and more approachable (streamlit#6679)
  README updated with links to docs (streamlit#6780)
  Refactor: withHostCommunication -> HostCommunicationManager (streamlit#6746)
  Add python 3.11 to classifiers list. (streamlit#6786)
  Release/1.23.1 (streamlit#6777)
  We depend on typing-extensions 4.0.1 (streamlit#6776)
  Release/1.23.0 (streamlit#6773)
  Fix typo in data_editor docstring and deprecation msg: `edited_rows` -> `edited_cells` (streamlit#6770)
  Fix: Remove flaky date input calendar snapshot test (streamlit#6769)
  Fix 3218, patch pydantic in_ipython function (streamlit#6664)
  Upgrade react-range to fix memory usage of sliders (streamlit#6764)
  Make st.write pretty-print dataclasses using st.help (streamlit#6750)
  Replace curly with straightquotes in docstring examples (streamlit#6757)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change:chore PR contains maintenance or housekeeping change impact:internal PR changes only affect internal code security-assessment-completed Security assessment has been completed for PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants