-
-
Notifications
You must be signed in to change notification settings - Fork 546
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
ocr-numbers: implement error object #1335
Conversation
@@ -1,6 +1,6 @@ | |||
{ | |||
"exercise": "ocr-numbers", | |||
"version": "1.1.0", | |||
"version": "1.2.0", | |||
"comments": [ | |||
"An expectation of -1 indicates that a failure should be raised" |
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.
Redundant comment.
@@ -57,7 +57,7 @@ | |||
" " | |||
] | |||
}, | |||
"expected": -1 | |||
"expected": {"error": "must be a multiple of four"} |
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.
What must be?
@@ -54,7 +54,7 @@ | |||
" " | |||
] | |||
}, | |||
"expected": {"error": "must be a multiple of four"} | |||
"expected": {"error": "rows must be a multiple of four"} |
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.
This doesn't match the language of the description
@@ -67,7 +67,7 @@ | |||
" " | |||
] | |||
}, | |||
"expected": {"error": "must be a multiple of three"} | |||
"expected": {"error": "rows must be a multiple of three"} |
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.
This has a different meaning to the description.
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.
Make the error messages match the descriptions
per exercism#1311 ocr-numbers: remove redundant comment per @Insti's suggestion. ocr-numbers: add clarity to error messages per @Insti suggestion. ocr-numbers: improved wording of error messages
4327861
to
965ecad
Compare
per #1311