Skip to content

Docstrings for 1.54.0#13804

Merged
sfc-gh-dmatthews merged 6 commits intodevelopfrom
docs/1.54.0
Feb 3, 2026
Merged

Docstrings for 1.54.0#13804
sfc-gh-dmatthews merged 6 commits intodevelopfrom
docs/1.54.0

Conversation

@sfc-gh-dmatthews
Copy link
Copy Markdown
Contributor

@sfc-gh-dmatthews sfc-gh-dmatthews commented Feb 3, 2026

Describe your changes

Small update to logo.py and config.py for style.

Screenshot or video (only for visual changes)

GitHub Issue Link (if applicable)

Testing Plan

  • Explanation of why no additional tests are needed
  • 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.

Copilot AI review requested due to automatic review settings February 3, 2026 19:08
@snyk-io
Copy link
Copy Markdown
Contributor

snyk-io bot commented Feb 3, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@sfc-gh-dmatthews sfc-gh-dmatthews added security-assessment-completed impact:internal PR changes only affect internal code change:docs PR contains documentation change labels Feb 3, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 3, 2026

✅ PR preview is ready!

Name Link
📦 Wheel file https://core-previews.s3-us-west-2.amazonaws.com/pr-13804/streamlit-1.53.1-py3-none-any.whl
📦 @streamlit/component-v2-lib Download from artifacts
🕹️ Preview app pr-13804.streamlit.app (☁️ Deploy here if not accessible)

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates user-facing documentation strings for configuration and st.logo to improve clarity and consistency.

Changes:

  • Refined the client.showErrorLinks option description in config.py.
  • Updated st.logo docstring parameter descriptions in logo.py.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
lib/streamlit/config.py Updates the client.showErrorLinks config option description text.
lib/streamlit/commands/logo.py Adjusts st.logo docstring parameter descriptions for image and icon_image.

sfc-gh-dmatthews and others added 2 commits February 3, 2026 11:21
Co-authored-by: graphite-app[bot] <96075541+graphite-app[bot]@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@sfc-gh-nbellante sfc-gh-nbellante left a comment

Choose a reason for hiding this comment

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

👍

@sfc-gh-dmatthews sfc-gh-dmatthews merged commit b8eb947 into develop Feb 3, 2026
41 checks passed
@sfc-gh-dmatthews sfc-gh-dmatthews deleted the docs/1.54.0 branch February 3, 2026 21:33
github-actions bot pushed a commit that referenced this pull request Feb 3, 2026
## Describe your changes
Small update to logo.py and config.py for style.

## Screenshot or video (only for visual changes)

## GitHub Issue Link (if applicable)

## Testing Plan

- Explanation of why no additional tests are needed
- 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.

---------

Co-authored-by: graphite-app[bot] <96075541+graphite-app[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change:docs PR contains documentation change impact:internal PR changes only affect internal code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants