-
Notifications
You must be signed in to change notification settings - Fork 4
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
Added data object documentation #298
Conversation
Code Coverage Summary
Diff against main
Results for commit: 398b393 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Unit Tests Summary 1 files 40 suites 5m 54s ⏱️ Results for commit 398b393. ♻️ This comment has been updated with latest results. |
Unit Test Performance Difference
Additional test case details
Results for commit bb75327 ♻️ This comment has been updated with latest results. |
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.
Thanks @gowerc , reading through this very useful overview I got some questions on how to make 1) variable naming and 2) terms in general more consistent, please see below - applies potentially to other things down the list too.
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 much better! thanks @gowerc
Closes #280