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

chore(developer): rename messages.ts for clarity #9920

Merged
merged 1 commit into from
Nov 2, 2023

Conversation

mcdurdin
Copy link
Member

@mcdurdin mcdurdin commented Nov 1, 2023

Renames two messages.ts to kmn-compiler-messages.ts and kmw-compiler-messages.ts, as otherwise confusion reigns.

Also a minor cleanup of the error messages from touch layout processing, to clarify message names and return appropriate message for missing layout file.

@keymanapp-test-bot skip

Renames two messages.ts to kmn-compiler-messages.ts and
kmw-compiler-messages.ts, as otherwise confusion reigns.

Also a minor cleanup of the error messages from touch layout processing,
to clarify message names and return appropriate message for missing
layout file.
@mcdurdin mcdurdin requested a review from darcywong00 as a code owner November 1, 2023 02:32
@keymanapp-test-bot
Copy link

keymanapp-test-bot bot commented Nov 1, 2023

User Test Results

Test specification and instructions

User tests are not required

Test Artifacts

@@ -42,7 +42,7 @@ export const enum KmnCompilerMessageRanges {

/*
The messages in this class share the namespace with messages from
kmn_compiler_errors.h, which are generated by kmcmplib, so values below 0x1000
kmn_compiler_errors.h, which are generated by kmcmplib, so values below 0x900
Copy link
Contributor

Choose a reason for hiding this comment

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

Why did this change?

Copy link
Contributor

@rc-swag rc-swag left a comment

Choose a reason for hiding this comment

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

lgtm

Base automatically changed from fix/developer/7880-error-if-virtual-key-in-context to master November 2, 2023 04:05
@mcdurdin mcdurdin merged commit 328f808 into master Nov 2, 2023
3 checks passed
@mcdurdin mcdurdin deleted the chore/developer/rename-compiler-messages branch November 2, 2023 04:06
@keyman-server
Copy link
Collaborator

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

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.

3 participants