-
Notifications
You must be signed in to change notification settings - Fork 44
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 output generation for device validation reports #400
Conversation
return errors.size() == 1 ? makeEntry(errors.get(0)) : makeCompoundEntry(errors); | ||
} | ||
|
||
private Entry makeCompoundEntry(List<Exception> erx) { |
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.
Could we have docstrings that give example input/output here? github doesn't lend itself to jumping around very quickly to piece it together.
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.
Done -- added javadoc to ~4 or so of the methods relating to this.
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.
I think at some point we need a util class that has all these general purpose functions like exception translators etc? But for now lgtm
Things in validator/trace/simple/ can be ignored (trace input files)
Things in validator/trace/simple.out/ are examples of what the output produced is, so should be checked for completeness.
Other code bits deserve code-review as per normal.