Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove compatibility symbols (Proposal 19) #14

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

MDLC01
Copy link
Collaborator

@MDLC01 MDLC01 commented Nov 29, 2024

Description

This PR implements Proposal 19 (iteration 1 iteration 2) from the Symbol Proposals document. It removes compatibility codepoints. For some context, see the Unit Symbols paragraphs of section 22.2.1 of the Unicode 16.0.0 Core Specification, as well as the description of Proposal 19 in the aforementioned Symbol Proposals document (copied below for simplicity).

Proposal 19 (iteration 2).

Breaking changes (cc. @laurmaedje)

angle.l.double and angle.r.double change codepoints. degree.c, degree.f, ohm, ohm.inv, angstrom, and kelvin become invalid.

@MDLC01 MDLC01 added the breaking This involves a breaking change label Nov 29, 2024
@MDLC01
Copy link
Collaborator Author

MDLC01 commented Nov 30, 2024

Regarding U+2127 ℧, which this renames to mho, I'm thinking it could make sense to add it as Omega.inv in addition (or in place of) mho.

Screenshot of the paragraph on ℧ in the Symbol Proposals document

@laurmaedje
Copy link
Member

I'm okay with removing the compatibility symbols.

@mkorje
Copy link
Collaborator

mkorje commented Dec 12, 2024

I agree we should add Omega.inv, and I'd argue in place of mho since I imagine a user might expect ohm to also work then. But as mho is a Unicode informative alias and I don't know anything about its use, I'm not quite sure if omitting mho would be good or bad.

@dccsillag
Copy link
Collaborator

dccsillag commented Dec 12, 2024 via email

@MDLC01
Copy link
Collaborator Author

MDLC01 commented Dec 12, 2024

I replaced mho with Omega.inv, as "mho" seems to mostly be an outdated unit.1 Of course, we can always add mho later if necessary.

Footnotes

  1. https://en.wikipedia.org/wiki/Siemens_(unit)#Mho

@Enivex
Copy link
Collaborator

Enivex commented Dec 12, 2024

It's so cute though!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking This involves a breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants