Skip to content

Conversation

@lukasmasuch
Copy link
Collaborator

@lukasmasuch lukasmasuch commented Jan 9, 2025

Describe your changes

We use a set of all existing emojis to validate emoji usage across various elements. This collection needs routine updates whenever new emojis are released.

This PR also includes a small fix to get the update_emojis.py script running again with the latest version of emoji library.

Emojis before update: 5034
Emojis after update: 5042


Contribution License Agreement

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

@lukasmasuch lukasmasuch added security-assessment-completed Security assessment has been completed for PR change:other PR contains other type of change impact:users PR changes affect end users labels Jan 9, 2025
Copy link
Contributor

@sfc-gh-pchiu sfc-gh-pchiu left a comment

Choose a reason for hiding this comment

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

🆕 ✅

@lukasmasuch lukasmasuch merged commit db19634 into develop Jan 10, 2025
44 of 47 checks passed
@lukasmasuch lukasmasuch deleted the update-emoji-list branch January 10, 2025 12:49
edegp pushed a commit to edegp/streamlit that referenced this pull request Jan 19, 2025
## Describe your changes

We use a set of all existing emojis to validate emoji usage across
various elements. This collection needs routine updates whenever new
emojis are released.

This PR also includes a small fix to get the `update_emojis.py` script
running again with the latest version of `emoji` library.

Emojis before update: 5034
Emojis after update:  5042

---

**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:other PR contains other type of change impact:users PR changes affect end users security-assessment-completed Security assessment has been completed for PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants