-
-
Notifications
You must be signed in to change notification settings - Fork 103
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
chore(core): km_core_cp -> km_core_cu #11341
Conversation
- km_core_cp represents a 16 bit code unit, not a code point. Fixes: #11033
User Test ResultsTest specification and instructions User tests are not required Test Artifacts
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change makes sense and clarifies that we're dealing with code units, not code points. Unfortunately this requires an API change, and since we're renaming an API method a API version bump.
- fix comment - fix API version Fixes: #11033 Co-authored-by: Eberhard Beilharz <ermshiperete@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM -- aside from chgs that @ermshiperete requested
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I still think we need to bump the API version
- also minor typo in docs
OK.. i think I did that now.. is it correct? |
Failed Developer build:
This causes a cascade of file naming issues -- need to update build scripts in Keyman Developer and dependencies in tike itself. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM on the Linux side.
…m/keymanapp/keyman into chore/core/11033-points-not-units
am I the first to rename this file? I suppose so since it goes from 1 to 2. See if this update works - once it works, I'll also update the docs to make the pieces easier to find. |
Co-authored-by: Eberhard Beilharz <ermshiperete@users.noreply.github.com>
Changes in this pull request will be available for download in Keyman version 18.0.32-alpha |
Fixes: #11033
@keymanapp-test-bot skip