Skip to content

Conversation

@AnOctopus
Copy link
Contributor

Moving the package install to before proto generation should ensure we have the needed mypy proto package installed. Don't know how it passed previously.

Testing Plan

  • Explanation of why no additional tests are needed
    CI change
  • Unit Tests (JS and/or Python)
  • E2E Tests
  • Any manual testing needed?

Contribution License Agreement

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

This should ensure we have the needed mypy proto package installed.
Don't know how it passed previously.
@AnOctopus AnOctopus added security-assessment-completed Security assessment has been completed for PR impact:internal PR changes only affect internal code change:bugfix PR contains bug fix implementation labels Jun 27, 2023
@AnOctopus AnOctopus merged commit 585d1ca into streamlit:develop Jun 27, 2023
@AnOctopus AnOctopus deleted the fix/min-dep-testing-protobuf branch June 27, 2023 17:59
tconkling added a commit to tconkling/streamlit that referenced this pull request Jun 27, 2023
* develop:
  Fix protobuf gen in min deps workflow (streamlit#6909)
asmeralt pushed a commit to asmeralt/streamlit that referenced this pull request Sep 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change:bugfix PR contains bug fix implementation 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