Skip to content

Improve phrasing of st.plotly_chart **kwarg deprecation warning #12989

Merged
sfc-gh-tteixeira merged 2 commits intodevelopfrom
plotly_warning
Nov 11, 2025
Merged

Improve phrasing of st.plotly_chart **kwarg deprecation warning #12989
sfc-gh-tteixeira merged 2 commits intodevelopfrom
plotly_warning

Conversation

@sfc-gh-tteixeira
Copy link
Copy Markdown
Contributor

Describe your changes

Improve the phrasing of the deprecation warning that's shown when users use variable keyword arguments in st.plotly_chart.

Screenshot or video (only for visual changes)

n/a

GitHub Issue Link (if applicable)

n/a

Testing Plan

  • Explanation of why no additional tests are needed: this is just a simple copy 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.

@sfc-gh-tteixeira sfc-gh-tteixeira enabled auto-merge (squash) November 10, 2025 23:10
@snyk-io
Copy link
Copy Markdown
Contributor

snyk-io bot commented Nov 10, 2025

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

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

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

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Nov 10, 2025

✅ PR preview is ready!

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

@sfc-gh-lmasuch sfc-gh-lmasuch added security-assessment-completed change:other PR contains other type of change impact:users PR changes affect end users labels Nov 11, 2025
Copy link
Copy Markdown
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 👍

@sfc-gh-tteixeira sfc-gh-tteixeira merged commit cbe26ad into develop Nov 11, 2025
50 of 52 checks passed
@sfc-gh-tteixeira sfc-gh-tteixeira deleted the plotly_warning branch November 11, 2025 01:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change:other PR contains other type of change impact:users PR changes affect end users

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants