[api.keyman.com] Lexical model schema drafts#1
Conversation
mcdurdin
force-pushed
the
lexical-model-schemas
branch
from
858f8d8 to
7807ed1
Compare
mcdurdin
dismissed
darcywong00’s stale review
Source model_info was broken, needs re-review.
|
I'm assuming some of the string patterns in |
| "version": { | ||
| "type": "string" | ||
| }, | ||
| "rtl": { |
There was a problem hiding this comment.
Should this be isRTL to be consistent with #/definitions/ModelInfo?
There was a problem hiding this comment.
Consistency would be a nice thing to have. Sadly I've chosen to use rtl to be consistent with package.json#/definitions/keyboards/rtl... so at least the fields are relatively consistent within a single file.
I'm not sure which string patterns you mean? If you mean the filename patterns, I don't plan to relax them because they are enforcing a structure in the repository and distribution. There's no requirement for models distributed outside our repo/Keyman Cloud distribution model to follow those patterns, but then they don't need a .model_info file either. Hope that makes sense? Note: I realised I annotated the 11.0 version of the kmp.json metadata file. Will correct to annotating 12.0 version (requires a separate PR to seed the 12.0 docs). |
…ts for model_info
|
Not merging yet: need to sort out versioning on (all) schema files |
3 participants
No description provided.