Skip to content

Copy entire .streamlit directory instead of single config file#329

Merged
t0mdavid-m merged 1 commit intomainfrom
claude/copy-streamlit-docker-XcH1j
Jan 29, 2026
Merged

Copy entire .streamlit directory instead of single config file#329
t0mdavid-m merged 1 commit intomainfrom
claude/copy-streamlit-docker-XcH1j

Conversation

@t0mdavid-m
Copy link
Member

@t0mdavid-m t0mdavid-m commented Jan 29, 2026

Summary

Updated Docker configuration to copy the entire .streamlit/ directory instead of just the config.toml file. This allows for more flexible Streamlit configurations and ensures all files in the directory are included in the Docker image.

Changes

  • Modified Dockerfile to copy .streamlit/ directory recursively instead of just .streamlit/config.toml
  • Modified Dockerfile_simple with the same change for consistency
  • This enables inclusion of additional Streamlit configuration files (e.g., secrets.toml, custom theme files) if needed in the future

Implementation Details

  • Changed COPY .streamlit/config.toml /app/.streamlit/config.toml to COPY .streamlit/ /app/.streamlit/
  • Applied to both Dockerfile variants to maintain consistency across build configurations
  • The change is backward compatible as config.toml will still be copied as part of the directory

https://claude.ai/code/session_01VQFwSZJHdHDfhfQxF4FqiQ

Summary by CodeRabbit

  • Chores
    • Updated container build configuration to include all Streamlit settings files, ensuring complete configuration is available in the deployed application.

✏️ Tip: You can customize this high-level summary in your review settings.

Previously only .streamlit/config.toml was copied. This change copies
the entire .streamlit folder including credentials.toml and
secrets.toml.example files.

https://claude.ai/code/session_01VQFwSZJHdHDfhfQxF4FqiQ
@t0mdavid-m t0mdavid-m merged commit 45518a1 into main Jan 29, 2026
4 of 7 checks passed
@coderabbitai
Copy link

coderabbitai bot commented Jan 29, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Both Dockerfile and Dockerfile_simple were modified to copy the entire .streamlit directory into the container rather than just the config.toml file. This change broadens the set of configuration files available in the Docker image at runtime.

Changes

Cohort / File(s) Summary
Docker Build Configuration
Dockerfile, Dockerfile_simple
Expanded the Streamlit configuration copy step from targeting a single config.toml file to copying the entire .streamlit/ directory, including all contained files.

Poem

🐰 A rabbit hops through Docker's land,
Where config files once barely spanned,
But now the whole .streamlit nest,
Gets copied in—now that's the best!
More configs to play with today, 🎉


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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