Skip to content

Improve invalid configuration error messages #6240

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

Merged
merged 1 commit into from
Mar 1, 2018

Conversation

theotherjimmy
Copy link
Contributor

@theotherjimmy theotherjimmy commented Feb 28, 2018

old

[ERROR] OrderedDict([(u'help', u'Address of the active application firmware in flash'), (u'value', u'POST_APPLICATION_ADDR'), (u'required', 1)]) is not valid under any of the given schemas,OrderedDict([(u'help', u'Flash sector address for SOTP sector 1'), (u'macro_name', u'PAL_INTERNAL_FLASH_SECTION_1_ADDRESS'), (u'value', [])]) is not valid under any of the given schemas
[ERROR] 1 is not of type u'string'

new

[ERROR] in ./mbed_app.json element config.application-start-address.required: 1 is not of type u'boolean'; in ./mbed_app.json element config.sotp-section-1-address.value: [] is not of type u'integer', u'string', u'boolean', u'null'                                                  
[ERROR] in ./tools/mbed_lib.json element name: 1 is not of type u'string'

bug fix/ui improvement?

@marcuschangarm

@theotherjimmy theotherjimmy changed the title /home/jimmy/pr-message.md Add BL parameter definitions as new document Improve invalid configuration error messages Feb 28, 2018
@theotherjimmy theotherjimmy changed the title Add BL parameter definitions as new document Improve invalid configuration error messages Improve invalid configuration error messages Feb 28, 2018
@theotherjimmy
Copy link
Contributor Author

@cmonr Verified working with app and lib config. Could you review?

@cmonr
Copy link
Contributor

cmonr commented Feb 28, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Feb 28, 2018

Build : SUCCESS

Build number : 1308
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/6240/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented Mar 1, 2018

@mbed-ci
Copy link

mbed-ci commented Mar 1, 2018

@adbridge adbridge merged commit f624802 into ARMmbed:master Mar 1, 2018
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.

4 participants