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(resources,common,developer,core): ldml update out of techpreview 🙀 #10913

Merged
merged 10 commits into from
Mar 6, 2024

Conversation

srl295
Copy link
Member

@srl295 srl295 commented Mar 1, 2024

  • includes patch for bn.xml
  • pre-release ldml
  • update to cldr 40fa7fd5b074a203581d04ca0c0b004ab9aa491b (main)

Fixes: #10900
Fixes: #10901

(Review note: split into 4 commits by functional area)

@keymanapp-test-bot skip

@keymanapp-test-bot keymanapp-test-bot bot added this to the B17S2 milestone Mar 1, 2024
@srl295 srl295 self-assigned this Mar 1, 2024
@srl295 srl295 marked this pull request as draft March 1, 2024 17:44
@srl295 srl295 force-pushed the chore/resources/10900-ldml-45 branch from 0c9ef49 to f21e675 Compare March 3, 2024 04:37
@srl295 srl295 marked this pull request as ready for review March 3, 2024 04:37
Comment on lines +32 to +33
"$KEYMAN_ROOT/resources/standards-data/ldml-keyboards/45/ldml-keyboard3.schema.json"
"$KEYMAN_ROOT/resources/standards-data/ldml-keyboards/45/ldml-keyboardtest3.schema.json"
Copy link
Member Author

Choose a reason for hiding this comment

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

100% kicking the 'what about 46' can down the road here

srl295 added 4 commits March 2, 2024 22:40
- update to cldr 40fa7fd5b074a203581d04ca0c0b004ab9aa491b (main) - alpha1+
- includes patch for bn

Fixes: #10900
- updates to support techpreview -> 45

Fixes: #10900
- updates to support techpreview -> 45

Fixes: #10900
- updates to support techpreview -> 45
- test changes

Fixes: #10900
@srl295 srl295 force-pushed the chore/resources/10900-ldml-45 branch from f21e675 to 17a653f Compare March 3, 2024 04:40
@mcdurdin mcdurdin modified the milestones: B17S2, B17S3 Mar 3, 2024
@mcdurdin
Copy link
Member

mcdurdin commented Mar 3, 2024

9 failing tests:

12:50:29     125 passing (950ms)
12:50:29     1 pending
12:50:29     9 failing
12:50:29   
12:50:29     1) disp
12:50:28          should fail to compile: sections/disp/invalid-var.xml:
12:50:28   
12:50:28         expected errors to be included
12:50:28         + expected - actual
12:50:28   
12:50:28          [
12:50:28            {
12:50:28         -    "code": 3149825
12:50:28         +    "code": 3145756
12:50:28              "detail": [undefined]
12:50:28         -    "message": "Error validating LDML XML file: /keyboard3: required: must have required property 'conformsTo' missingProperty=\"conformsTo\""
12:50:28         +    "message": "Reference to undefined string variable: ${missingdisplay}"
12:50:28            }
12:50:28         +  {
12:50:28         +    "code": 3145756
12:50:28         +    "detail": [undefined]
12:50:28         +    "message": "Reference to undefined string variable: ${missingoutput}"
12:50:28         +  }
12:50:28          ]
12:50:28   
12:50:28         at Context.<anonymous> (file:///C:/BuildAgent/work/7ac43416c45637e9/keyman/developer/src/kmc-ldml/test/helpers/index.ts:250:16)
12:50:28   
12:50:28     2) keys.kmap
12:50:28          should fail to compile: sections/keys/invalid-undefined-var-1.xml:
12:50:28   
12:50:28         expected errors to be included
12:50:28         + expected - actual
12:50:28   
12:50:28          [
12:50:28            {
12:50:28         -    "code": 3149825
12:50:28         +    "code": 3145756
12:50:28              "detail": [undefined]
12:50:28         -    "message": "Error validating LDML XML file: /keyboard3: required: must have required property 'conformsTo' missingProperty=\"conformsTo\""
12:50:28         +    "message": "Reference to undefined string variable: ${varsok}"
12:50:28            }
12:50:28          ]
12:50:28   
12:50:28         at Context.<anonymous> (file:///C:/BuildAgent/work/7ac43416c45637e9/keyman/developer/src/kmc-ldml/test/helpers/index.ts:250:16)
12:50:28   
12:50:28     3) keys.kmap
12:50:28          should fail to compile: sections/keys/invalid-undefined-var-1b.xml:
12:50:28   
12:50:28         expected errors to be included
12:50:28         + expected - actual
12:50:28   
12:50:28          [
12:50:28            {
12:50:28         -    "code": 3149825
12:50:28         +    "code": 3145756
12:50:28              "detail": [undefined]
12:50:28         -    "message": "Error validating LDML XML file: /keyboard3: required: must have required property 'conformsTo' missingProperty=\"conformsTo\""
12:50:28         +    "message": "Reference to undefined string variable: ${varsok}"
12:50:28            }
12:50:28          ]
12:50:28   
12:50:28         at Context.<anonymous> (file:///C:/BuildAgent/work/7ac43416c45637e9/keyman/developer/src/kmc-ldml/test/helpers/index.ts:250:16)
12:50:28   
12:50:28     4) tran
12:50:28          should fail to compile: sections/tran/fail-missing-var-1.xml:
12:50:28   
12:50:28         expected errors to be included
12:50:28         + expected - actual
12:50:28   
12:50:28          [
12:50:28            {
12:50:28         -    "code": 3149825
12:50:28         +    "code": 3145756
12:50:28              "detail": [undefined]
12:50:28         -    "message": "Error validating LDML XML file: /keyboard3: required: must have required property 'conformsTo' missingProperty=\"conformsTo\""
12:50:28         +    "message": "Reference to undefined string variable: ${missingfrom}"
12:50:28            }
12:50:28          ]
12:50:28   
12:50:28         at Context.<anonymous> (file:///C:/BuildAgent/work/7ac43416c45637e9/keyman/developer/src/kmc-ldml/test/helpers/index.ts:250:16)
12:50:28   
12:50:28     5) tran
12:50:28          should fail to compile: sections/tran/fail-missing-var-2.xml:
12:50:28   
12:50:28         expected errors to be included
12:50:28         + expected - actual
12:50:28   
12:50:28          [
12:50:28            {
12:50:28         -    "code": 3149825
12:50:28         +    "code": 3145756
12:50:28              "detail": [undefined]
12:50:28         -    "message": "Error validating LDML XML file: /keyboard3: required: must have required property 'conformsTo' missingProperty=\"conformsTo\""
12:50:28         +    "message": "Reference to undefined string variable: ${missingto}"
12:50:28            }
12:50:28          ]
12:50:28   
12:50:28         at Context.<anonymous> (file:///C:/BuildAgent/work/7ac43416c45637e9/keyman/developer/src/kmc-ldml/test/helpers/index.ts:250:16)
12:50:28   
12:50:28     6) tran
12:50:28          should fail to compile: sections/tran/fail-missing-var-3.xml:
12:50:28   
12:50:28         expected errors to be included
12:50:28         + expected - actual
12:50:28   
12:50:28          [
12:50:28            {
12:50:28         -    "code": 3149825
12:50:28         +    "code": 3145757
12:50:28              "detail": [undefined]
12:50:28         -    "message": "Error validating LDML XML file: /keyboard3: required: must have required property 'conformsTo' missingProperty=\"conformsTo\""
12:50:28         +    "message": "Reference to undefined set variable: $[missingset]"
12:50:28            }
12:50:28          ]
12:50:28   
12:50:28         at Context.<anonymous> (file:///C:/BuildAgent/work/7ac43416c45637e9/keyman/developer/src/kmc-ldml/test/helpers/index.ts:250:16)
12:50:28   
12:50:28     7) tran
12:50:28          should fail to compile: sections/tran/fail-missing-var-4.xml:
12:50:28   
12:50:28         expected errors to be included
12:50:28         + expected - actual
12:50:28   
12:50:28          [
12:50:28            {
12:50:28         -    "code": 3149825
12:50:28         +    "code": 3145757
12:50:28              "detail": [undefined]
12:50:28         -    "message": "Error validating LDML XML file: /keyboard3: required: must have required property 'conformsTo' missingProperty=\"conformsTo\""
12:50:28         +    "message": "Reference to undefined set variable: $[missingset]"
12:50:28            }
12:50:28          ]
12:50:28   
12:50:28         at Context.<anonymous> (file:///C:/BuildAgent/work/7ac43416c45637e9/keyman/developer/src/kmc-ldml/test/helpers/index.ts:250:16)
12:50:28   
12:50:28     8) tran
12:50:28          should fail to compile: sections/tran/fail-missing-var-5.xml:
12:50:28   
12:50:28         expected errors to be included
12:50:28         + expected - actual
12:50:28   
12:50:28          [
12:50:28            {
12:50:28         -    "code": 3149825
12:50:28         +    "code": 3145757
12:50:28              "detail": [undefined]
12:50:28         -    "message": "Error validating LDML XML file: /keyboard3: required: must have required property 'conformsTo' missingProperty=\"conformsTo\""
12:50:28         +    "message": "Reference to undefined set variable: $[missingset]"
12:50:28            }
12:50:28          ]
12:50:28   
12:50:28         at Context.<anonymous> (file:///C:/BuildAgent/work/7ac43416c45637e9/keyman/developer/src/kmc-ldml/test/helpers/index.ts:250:16)
12:50:28   
12:50:28     9) tran
12:50:28          should fail to compile: sections/tran/fail-missing-var-6.xml:
12:50:28   
12:50:28         expected errors to be included
12:50:28         + expected - actual
12:50:28   
12:50:28          [
12:50:28            {
12:50:28         -    "code": 3149825
12:50:28         +    "code": 3145756
12:50:28              "detail": [undefined]
12:50:28         -    "message": "Error validating LDML XML file: /keyboard3: required: must have required property 'conformsTo' missingProperty=\"conformsTo\""
12:50:28         +    "message": "Reference to undefined string variable: ${missingstr}"
12:50:28            }
12:50:28          ]
12:50:28   
12:50:28         at Context.<anonymous> (file:///C:/BuildAgent/work/7ac43416c45637e9/keyman/developer/src/kmc-ldml/test/helpers/index.ts:250:16)
12:50:28   
12:50:28   

Copy link
Member

@mcdurdin mcdurdin left a comment

Choose a reason for hiding this comment

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

Apart from the failing tests, a small update to kmconvert should also address #10901.

@srl295
Copy link
Member Author

srl295 commented Mar 4, 2024

hrm, i think on the third try of generating this i ran kmc/build.sh and missed kmc-ldml/build.sh.. tests are doing their job!

@srl295 srl295 requested a review from mcdurdin March 4, 2024 15:52
@srl295 srl295 merged commit dfbdfff into beta Mar 6, 2024
25 checks passed
@srl295 srl295 deleted the chore/resources/10900-ldml-45 branch March 6, 2024 01:26
@keyman-server
Copy link
Collaborator

Changes in this pull request will be available for download in Keyman version 17.0.283-beta

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants