Skip to content

Conversation

karangattu
Copy link
Collaborator

@karangattu karangattu commented Sep 11, 2025

This pull request adds the pydantic library as a dependency to several test application requirements files across the codebase. This ensures that all relevant test apps have consistent dependencies, likely in preparation for or support of features that require pydantic.

Dependency updates:

  • Added pydantic to the following requirements files to standardize dependencies for test applications:
    • express-accordion/app_requirements.txt
    • express-dataframe/app_requirements.txt
    • express-folium/app_requirements.txt
    • express-page_default/app_requirements.txt
    • express-page_fillable/app_requirements.txt
    • express-page_fluid/app_requirements.txt
    • express-page_sidebar/app_requirements.txt
    • shiny-client-console-error/app_requirements.txt
    • plotly/app_requirements.txt

karangattu and others added 6 commits September 11, 2025 08:46
Bump shiny to v1.4.0 and htmltools to v0.6.0 in the deploy-tests workflow for GitHub installs. This ensures tests run against the latest versions from the respective repositories.
Added 'pydantic' to the app_requirements.txt files for several Playwright deployment test apps to ensure consistent dependency management and support for features requiring pydantic.
The deploy tests workflow will no longer run on pushes to the 'use-latest-version' branch. This streamlines workflow triggers to only relevant branches.
@karangattu karangattu changed the title fix(deploys): Update GitHub shiny and htmltools versions in CI fix(deploys): Add pydantic as a dependency in the requirements.txt files for deployed apps Sep 11, 2025
@cpsievert
Copy link
Collaborator

Thanks for investigating @karangattu. I just made a fix for this in shinychat and did another release. Will trigger tests to run on main 🤞

@cpsievert cpsievert closed this Sep 11, 2025
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