-
Notifications
You must be signed in to change notification settings - Fork 20
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
fix: field extraction error message #448
Conversation
1346a70
to
21c03da
Compare
|
||
assert datamodel_check and field_extraction_check, ( | ||
f" Issue with either field extraction or data model.\nsearch={search}\n" | ||
f" data model check: {datamodel_check} \n" | ||
f" data model in requirement file {model_datalist}\n " | ||
f" data model extracted by TA {list(datamodel_based_on_tag.keys())}\n" | ||
f" Field_extraction_check: {field_extraction_check} \n" | ||
f" Key value not extracted by TA: {missing_key_value} \n" | ||
f" Mismatched key value: {mismapped_key_value_pair}\n" | ||
f" Field extraction errors: {json.dumps(missing_key_value, indent=4)}q \n" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo: "q" in the end
21c03da
to
3b24a1f
Compare
Expected release notes (by @nandinivij) fixes:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
🎉 This PR is included in version 1.10.1 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Please review
Changed the field messaging format to display errors as -