-
Notifications
You must be signed in to change notification settings - Fork 385
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
CLDR-17141 kbd: keep transform=no but document it #3332
CLDR-17141 kbd: keep transform=no but document it #3332
Conversation
I'd prefer to drop transform=no and simplify the mental model for keyboard authors. transform=no may seem simpler, but the caveats mean they have to have some grasp of transforms anyway, and then it just becomes an additional layer to remember. |
@jahorton fyi |
6ee3b50
to
00b938b
Compare
- don't name the special marker
ee8dc9c
to
32fe1fd
Compare
Notice: the branch changed across the force-push!
~ Your Friendly Jira-GitHub PR Checker Bot |
@miloush thoughts here? @mcdurdin requesting to drop (meaning i can just remove this PR). I agree that the complexity and limitations of
As "syntactic sugar" it may not add a lot. Probably better to work on documentation, and tooling (including debugging and test environments, test cases, etc) for transforms. |
Oh sorry I meant yes, feel free to drop. |
0ebdc9e
into
srl295/kbd/cldr-17141-drop-transform-no
@miloush sorry, i merged the wrong PR 😆 |
CLDR-17141
ALLOW_MANY_COMMITS=true