Skip to content

Conversation

@vdonato
Copy link
Collaborator

@vdonato vdonato commented Dec 14, 2022

📚 Context

It looks like the most recent CI failures affecting the release are caused by the pip-tools package.
This package stopped being used years ago (the last commit removing the usage of pip-compile
is from Jan 2019), and I'm fairly certain that it only stuck around in our Makefile afterwards because
we forgot to remove it.

@vdonato vdonato added the security-assessment-completed Security assessment has been completed for PR label Dec 14, 2022
@vdonato vdonato marked this pull request as ready for review December 14, 2022 23:56
@vdonato vdonato merged commit c59a138 into streamlit:develop Dec 15, 2022
@vdonato vdonato deleted the remove-pip-tools branch December 15, 2022 01:04
tconkling added a commit that referenced this pull request Dec 20, 2022
* develop:
  Change workers balancing logic for e2e tests to always cover all specs (#5865)
  Up version to 1.16.0 (#5852)
  Stop installing pip-tools (#5854)
  Add eslint no relative import paths plugin to pre commit hooks (#5839)
  Docs: move note to `experimental_allow_widgets` param description (#5843)
  Update element docstrings for colored text support (#5831)
  Add remark-directive plugin (Support for colored text in markdown) (#5774)
  Teach WebsocketConnection how to wait on an external auth token (#5728)
  Turn on streamlit theme for altair and plotly (#5796)
  Fix markdown headers spacing (#5829)
  Update error message and docstring in st.map (#5792)
  Remove unnecessarily methods from DeltaGenerator (#5824)
  Resolve vega-tooltip to later version (#5825)
  Easter Egg (#5817)
  Update Exception Layout to avoid overflow (#5700)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

security-assessment-completed Security assessment has been completed for PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants