Fixes #9692.

Signature checking was skipped because we missed a ".virtual" to force
nmake to build the test and test_i3633 targets. This opened up a small
cascade of related formatting issues on Makefiles, and the fact that the
test_i3633 (has there ever been a more poorly named project?) Makefile
did not even work.

Refactored significantly, added same tests to Developer Makefile, and
also now verifying the .msi and installer executable.

We can improve this further but I'd like to get this in to avoid further
critical issues with code signing given the current broken signing
configuration.

auto: increment master version to 17.0.187

feat(developer): warn if .kps includes a .js which is not touch-capable

fix(core): clean cached ICU in core

chore(developer): switch on code coverage reporting for kmc

Merge pull request #9658 from keymanapp/feat/developer/9325-hint-if-package-includes-keyboard-source-files

feat(developer): issue hint if package includes keyboard source files

chore(developer): reduce duplicate words warning to hint

feat(developer): show an INFO message when warnings have failed a build

Co-authored-by: Steven R. Loomis <srl295@gmail.com>

auto: increment master version to 17.0.186

feat(developer): ldml scan codes support 🙀 

Merge pull request #9661 from keymanapp/chore/developer/9317-test-for-check-filename-conventions-off

chore(developer): add test for `checkFilenameConventions == false` or unset

feat(developer): support `store(&version) '17.0'`

Fixes #9636.

chore(web): Add non-printing characters to the OSK

Fixes #9324.

As .js keyboards in packages are only used by the iOS and Android apps,
it's kinda helpful if the keyboard itself includes a touch layout. While
Keyman will fall back to a desktop vk layout if a touch layout is not
present, this is less than ideal -- but that's why this is setup as a
hint and not a warning.

Fixes #8661.

Fixes #9317.

- fixes to the special case test

Fixes: #9403

auto: increment master version to 17.0.185

- fix casing of k_oC1
- allow vkey 226 to be doubly defined
- drop LdmlVkeyNames table and test, not needed anymore (was for vkey #7135)

Fixes: #9403

Co-authored-by: Marc Durdin <marc@durdin.net>

Fixes #9325.

Also tweaks test for error_package_must_contain_a_model_or_a_keyboard as
the fixture had a .kmn, triggering the new hint, which was unhelpful in
this case.

Fixes #9541.

Also adds version 16.0 support to the kmcmplib compiler constants, and
unit tests for both versions. Does not add any support for automatic
version feature detection, because that forces an inverted dependency on
the touch layout compilation phase (done in kmc-kmw), which would be a
significant refactor. This may be something we need to support in the
future.

Fixes #9597.

Fixes #9610.

If a build fails when "treat warnings as errors" is on, but there are
only warnings and no error messages, it can be confusing for the
developer. Adds the following `INFO_WarningsHaveFailedBuild` message:

`The build failed because option "treat warnings as errors" is enabled
and there are one or more warnings.`

fix(web): fixes toolbar refocus timing after a keyboard change

- spec: 'hardware' is no longer an enum but a str.
Ripple effects here.
- custom scancodes and layouts are, interestingly, supported. But it's a warning.
- tests for bad scancodes.

Fixes: #9403

- also fix visual keyboard compiler to use same code

For: #9403

For: #9403