Report all issues rather than stop at first one. #450
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I created an empty list called "report". If constraints are satisfied, an error message is appended to the "report".
I updated all the conditions, and in the end, a dictionary is transformed into a string.
If that string is not empty, the error message is raised.
I used the "print" statement and "assert False" to check if errors are raised.
@janosg should I keep the "Invalid model specification:" message when the error is raised? All unit tests pass, so this should be good.