Skip to content

Conversation

@stesee
Copy link
Collaborator

@stesee stesee commented Oct 22, 2025

No description provided.

Copilot AI review requested due to automatic review settings October 22, 2025 05:28
Copy link
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

This PR updates the README documentation to clarify the sanitization rules implemented by the library. The changes provide more specific information about the supported functionality and include an additional example for better illustration.

  • Added clarification that the library handles "some unicode edge cases" in addition to Microsoft's documented rules
  • Enhanced the Unicode example in the compatibility table with a linked reference to emoji documentation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

README.md Outdated
| Unpaired Unicode surrogates | macOS, Linux | Windows | U+D800 - U+DFFF |
| NotAssigned to Unicode | macOS | Linux, Windows | U+67803, ... |
| "New" Unicode (today 17+) | macOS | Linux, Windows | 🫩 (U+1FAE9), ... |
| "New" Unicode (today 17+) | macOS | Linux, Windows | 🫩 [🫈](https://emojipedia.org/hairy-creature), ... |
Copy link

Copilot AI Oct 22, 2025

Choose a reason for hiding this comment

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

The example shows two different emojis (🫩 and 🫈) but the link text '🫈' doesn't match the first emoji shown (🫩). Consider making the link text match the first example emoji or adding separate links for clarity: 🫩 ([hairy-creature](https://emojipedia.org/hairy-creature)), ...

Suggested change
| "New" Unicode (today 17+) | macOS | Linux, Windows | 🫩 [🫈](https://emojipedia.org/hairy-creature), ... |
| "New" Unicode (today 17+) | macOS | Linux, Windows | [🫩](https://emojipedia.org/phoenix) [🫈](https://emojipedia.org/hairy-creature), ... |

Copilot uses AI. Check for mistakes.
@stesee stesee merged commit c92ddbe into main Oct 22, 2025
1 check passed
@github-actions github-actions bot locked and limited conversation to collaborators Oct 22, 2025
@stesee stesee deleted the stesee-patch-1 branch December 29, 2025 05:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants