Maldivian Rufiyaa Sign #1225
Draft
Maldivian Rufiyaa Sign #1225
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jowilco
requested review from
Ken-Whistler,
Copilot,
eggrobin,
josh-hadley and
markusicu
There was a problem hiding this comment.
Pull Request Overview
This PR adds the Maldivian Rufiyaa currency sign (₢) at code point U+20C2, following Unicode Consortium consensus decision [184-C18]. The change introduces a single new currency symbol character between the existing Saudi Riyal Sign (U+20C1) and UAE Dirham Sign (U+20C3).
Key Changes:
- Added U+20C2 RUFIYAA SIGN as a new currency symbol character
- Updated character property files to include the new code point in appropriate ranges
- Added comparison test file for validation against similar currency symbols
Reviewed Changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| AdditionComparisons/245.txt | Test file comparing RUFIYAA SIGN properties against SAUDI RIYAL SIGN |
| DerivedName.txt | Added character name entry for U+20C2 |
| DerivedLineBreak.txt | Consolidated currency range to include U+20C2 with PR (Prefix) line break class |
| DerivedGeneralCategory.txt | Removed U+20C2 from unassigned range, added to Sc (Currency_Symbol) category |
| DerivedEastAsianWidth.txt | Included U+20C2 in Neutral width currency range |
| DerivedCombiningClass.txt | Added U+20C2 to combining class 0 range |
| DerivedBidiClass.txt | Included U+20C2 in ET (European Number Terminator) bidi class |
| VerticalOrientation.txt | Added U+20C2 to R (Rotated) vertical orientation range |
| UnicodeData.txt | Added complete character entry for U+20C2 with all properties |
| Scripts.txt | Included U+20C2 in Common script range |
| LineBreak.txt | Consolidated currency range to include U+20C2 with PR line break property |
| EastAsianWidth.txt | Added U+20C2 to Neutral width currency range |
| DerivedCoreProperties.txt | Included U+20C2 in Grapheme_Base property range |
| DerivedAge.txt | Marked U+20C2 as newly assigned in Unicode 18.0 |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See https://github.com/unicode-org/sah/issues/632 and L2/25-187,
[184-C18] Consensus: Provisionally assign one code point for U+20C2 RUFIYAA SIGN in the Currency Symbols block as described in L2/25-122R. [Ref. 2.2 in L2/25-187]