Skip to content

change(developer): deprecate fix, clearcontext, bump max .kmx version to 19.0 - #16445

Merged
mcdurdin merged 2 commits into
masterfrom
change/developer/14211-deprecate-fix-clearcontext
Sep 1, 2026
Merged

change(developer): deprecate fix, clearcontext, bump max .kmx version to 19.0#16445
mcdurdin merged 2 commits into
masterfrom
change/developer/14211-deprecate-fix-clearcontext

Conversation

@mcdurdin

@mcdurdin mcdurdin commented Aug 27, 2026

Copy link
Copy Markdown
Member

Fixes: #14211
Test-bot: skip

@keymanapp-test-bot

keymanapp-test-bot Bot commented Aug 27, 2026

Copy link
Copy Markdown

User Test Results

Test specification and instructions

User tests are not required

Test Artifacts

  • Android
    • Keyman for Android apk - build : all tests passed (no artifacts on BuildLevel "build")
    • FirstVoices Keyboards for Android apk - build : all tests passed (no artifacts on BuildLevel "build")
    • KeyboardHarness apk - build : all tests passed (no artifacts on BuildLevel "build")
    • KMSample1 apk - build : all tests passed (no artifacts on BuildLevel "build")
    • KMSample2 apk - build : all tests passed (no artifacts on BuildLevel "build")
  • Developer
    • Keyman Developer - build : all tests passed (no artifacts on BuildLevel "build")
    • Compiler Regression Tests - build : all tests passed (no artifacts on BuildLevel "build")
    • kmcomp.zip - build : all tests passed (no artifacts on BuildLevel "build")
  • iOS
    • Keyman for iOS (simulator image) - build : all tests passed (no artifacts on BuildLevel "build")
    • FirstVoices Keyboards for iOS (simulator image) - build : all tests passed (no artifacts on BuildLevel "build")
  • Keyboards
    • Test Keyboards - build : all tests passed (no artifacts on BuildLevel "build")
  • macOS
    • Keyman for macOS (.dmg) - build : all tests passed (no artifacts on BuildLevel "build")
    • Keyman for macOS (.pkg) - build : all tests passed (no artifacts on BuildLevel "build")
  • Web
    • KeymanWeb Test Home - build : all tests passed (no artifacts on BuildLevel "build")
  • Windows
    • Keyman for Windows - build : all tests passed (no artifacts on BuildLevel "build")
    • FirstVoices Keyboards for Windows - build : all tests passed (no artifacts on BuildLevel "build")
    • Text Editor (ARM64) - build : all tests passed (no artifacts on BuildLevel "build")
    • Text Editor (x64) - build : all tests passed (no artifacts on BuildLevel "build")
    • Text Editor (x86) - build : all tests passed (no artifacts on BuildLevel "build")

@keymanapp-test-bot keymanapp-test-bot Bot added this to the A19S36 milestone Aug 27, 2026
@mcdurdin
mcdurdin requested review from ermshiperete and rc-swag and removed request for ermshiperete August 27, 2026 07:26
@mcdurdin mcdurdin changed the title refactor(windows): rename GetCapsAndNumlockState() to RefreshModifierState() change(developer): deprecate fix, clearcontext, bump max .kmx version to 19.0 Aug 27, 2026

@SabineSIL SabineSIL left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@keyman-server keyman-server modified the milestones: A19S36, B19S1 Aug 30, 2026
@mcdurdin

mcdurdin commented Sep 1, 2026

Copy link
Copy Markdown
Member Author

@rc-swag and @ermshiperete given the potential impact could I get a second review on this one please?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It's a bit confusing to have a file with deprecated in the name when it's actually not deprecated for v17...

Maybe add a comment in the file? Or name the file warn_deprecated_statement-clearcontext_not-deprecated-in-17.kmn ?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I'll add a comment as a follow-up (want to merge this chain before finishing up)

@mcdurdin
mcdurdin merged commit b2e72e9 into master Sep 1, 2026
32 checks passed
@mcdurdin
mcdurdin deleted the change/developer/14211-deprecate-fix-clearcontext branch September 1, 2026 15:32
@github-project-automation github-project-automation Bot moved this from Todo to Done in Keyman Sep 1, 2026
@keyman-server

Copy link
Copy Markdown
Collaborator

Changes in this pull request will be available for download in Keyman version 19.0.277-alpha

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

change(developer): mark fix and clearcontext as deprecated in existing compiler and remove CODE_CLEARCONTEXT

4 participants