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

fix(developer): enforce presence of kps Info.Description field in info compilers #12204

Merged

Conversation

mcdurdin
Copy link
Member

The Description field should be required for published keyboards and models, so this change means kmc-keyboard-info and kmc-model-info will report an error if it is missing.

Relates-to: keymanapp/keyboards#3037
Relates-to: keymanapp/lexical-models#262
Fixes: #12202

@keymanapp-test-bot skip

…o compilers

The Description field should be required for published keyboards and
models, so this change means kmc-keyboard-info and kmc-model-info will
report an error if it is missing.

Relates-to: keymanapp/keyboards#3037
Relates-to: keymanapp/lexical-models#262
Fixes: #12202
@mcdurdin mcdurdin requested a review from darcywong00 as a code owner August 16, 2024 06:06
@keymanapp-test-bot
Copy link

keymanapp-test-bot bot commented Aug 16, 2024

User Test Results

Test specification and instructions

User tests are not required

Test Artifacts

Copy link
Contributor

@darcywong00 darcywong00 left a comment

Choose a reason for hiding this comment

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

lgtm

@mcdurdin mcdurdin merged commit 5b8a05c into master Aug 16, 2024
7 checks passed
@mcdurdin mcdurdin deleted the fix/developer/12202-info-compilers-enforce-description-field branch August 16, 2024 13:12
@keyman-server
Copy link
Collaborator

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

mcdurdin added a commit that referenced this pull request Dec 8, 2024
…ard-info

Originally reported in #12202. Check was added in #12204, backported to
17.0.329 in #12207, but #12597 reports this for 17.0.331. So added test
to verify that it is working as expected (it is).

Note that this only applies when using the `--for-publishing` build flag
in kmc, because it is checked in the kmc-keyboard-info stage. Guessing
this is where the confusion arose.

Consolidated tests which used a common pattern for kmc-keyboard-info
calls. The remaining tests do stubs or calls into internal functions, so
they cannot be easily deduped.

Fixes: #12597
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug(developer): model info compiler does not always write description field
3 participants