Skip to content

Conversation

@jrieke
Copy link
Collaborator

@jrieke jrieke commented Nov 4, 2024

Describe your changes

This adds pathlib support for AppTest.from_file and st.components.v1.declare_component. Follow-up for #9711.

GitHub Issue Link (if applicable)

Testing Plan

Added Python unit tests for both functions.


Contribution License Agreement

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

@jrieke jrieke added security-assessment-completed Security assessment has been completed for PR change:feature PR contains new feature or enhancement implementation impact:users PR changes affect end users labels Nov 4, 2024
Copy link
Collaborator

@lukasmasuch lukasmasuch left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@jrieke jrieke merged commit 4cb3b2a into develop Nov 5, 2024
@jrieke jrieke deleted the feature/pathlib-support-follow-ups branch November 5, 2024 17:29
edegp pushed a commit to edegp/streamlit that referenced this pull request Jan 19, 2025
## Describe your changes

This adds pathlib support for `AppTest.from_file` and
`st.components.v1.declare_component`. Follow-up for streamlit#9711.

## GitHub Issue Link (if applicable)

## Testing Plan

Added Python unit tests for both functions.

---

**Contribution License Agreement**

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

---------

Co-authored-by: Johannes Rieke <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change:feature PR contains new feature or enhancement implementation impact:users PR changes affect end users security-assessment-completed Security assessment has been completed for PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants