Skip to content

[chore] Adding latest material icons#12473

Merged
sfc-gh-nbellante merged 3 commits intodevelopfrom
nico/update-icons-sept-9
Sep 9, 2025
Merged

[chore] Adding latest material icons#12473
sfc-gh-nbellante merged 3 commits intodevelopfrom
nico/update-icons-sept-9

Conversation

@sfc-gh-nbellante
Copy link
Copy Markdown
Contributor

@sfc-gh-nbellante sfc-gh-nbellante commented Sep 9, 2025

Describe your changes

Updating the latest material icons


Contribution License Agreement

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

@snyk-io
Copy link
Copy Markdown
Contributor

snyk-io bot commented Sep 9, 2025

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

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Sep 9, 2025

✅ PR preview is ready!

Name Link
📦 Wheel file https://core-previews.s3-us-west-2.amazonaws.com/pr-12473/streamlit-1.49.1-py3-none-any.whl
🕹️ Preview app pr-12473.streamlit.app (☁️ Deploy here if not accessible)

@sfc-gh-nbellante sfc-gh-nbellante marked this pull request as ready for review September 9, 2025 14:44
@sfc-gh-nbellante sfc-gh-nbellante changed the title Adding latest material icons [chore] Adding latest material icons Sep 9, 2025
@sfc-gh-nbellante sfc-gh-nbellante added security-assessment-completed change:chore PR contains maintenance or housekeeping change impact:users PR changes affect end users labels Sep 9, 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 👍 but needs an updated alert snapshot.

@sfc-gh-nbellante sfc-gh-nbellante enabled auto-merge (squash) September 9, 2025 15:18
@sfc-gh-nbellante sfc-gh-nbellante merged commit 70c464d into develop Sep 9, 2025
37 checks passed
@sfc-gh-nbellante sfc-gh-nbellante deleted the nico/update-icons-sept-9 branch September 9, 2025 17:43
sfc-gh-nbellante added a commit that referenced this pull request Sep 10, 2025
## Describe your changes

<!-- If it's a visual change, please include a screenshot or video! -->

This PR stemmed from
#12473 (review)

I noticed that the changed files workflow didn't fail the snapshots as I
expected. After talking with @lukasmasuch he said it's because it only
looks for changes to the _test.py files. This PR changes it so that it
will look at both.


## GitHub Issue Link (if applicable)

## Testing Plan

I tested it by changing the icon in st_alert.py and ensuring that the
workflow failed as expected.
<img width="1101" height="434" alt="image"
src="https://github.com/user-attachments/assets/6a0b16e2-9f58-4519-8623-facacac18dc1"
/>


---

**Contribution License Agreement**

By submitting this pull request you agree that all contributions to this
project are made under the Apache 2.0 license.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change:chore PR contains maintenance or housekeeping change impact:users PR changes affect end users

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants