-
Notifications
You must be signed in to change notification settings - Fork 2
Add category name and simplify test names #57
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
Conversation
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.
Looks good, will give it a test. I'm also thinking we should add an extended description which is collapsable (think we've discussed briefly before). can add it to the yaml file
Thanks! And/or link to the documentation where we describe the category in detail (in addition to each test, as we have at the moment). |
yeah i think would be good to have both, as people may not want to open loads of links and just have a quick glance. will barely take up any space when colapsed and should be simple to add |
just tested and works as expected! Also, sorting the cols appears to update the linked interactive plot correctly too which is great and didn't realise worked yet |
f44ce05
to
746362f
Compare
(Rebased) |
Pull Request Test Coverage Report for Build 17948956721Details
💛 - Coveralls |
Pre-review checklist for PR author
PR author must check the checkboxes below when creating the PR.
Summary
Adds yaml files for category title, used for the tab, title, etc., and description.
Also combined individual test names/titles into a single name parameter, as we don't need it to be extracted from the file name, and this allows more human-readable text.
Linked issue
Resolves #43
Testing
None
Could use testing from @joehart2001, and improvements to the category descriptions are welcome!