-
Notifications
You must be signed in to change notification settings - Fork 139
Fix failing load array test #1097
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
Codecov Report
@@ Coverage Diff @@
## main #1097 +/- ##
=======================================
Coverage 73.22% 73.22%
=======================================
Files 43 43
Lines 6424 6424
=======================================
Hits 4704 4704
Misses 1720 1720 |
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.
Some docstring comments
Co-authored-by: Doug Addy <da1910@users.noreply.github.com>
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.
Please see my notes regarding the DRY Principle. It's applicable for code and documentation.
Thank you @da1910 and @akaszynski for your reviews. @akaszynski Linking ... got it! |
Close #1061 by making sure the tested arrays have not all the elements equal.