-
Notifications
You must be signed in to change notification settings - Fork 61
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
Pytest Coverage: kp_norm #246
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #246 +/- ##
======================================
Coverage 96.4% 96.5%
======================================
Files 146 146
Lines 2956 2955 -1
Branches 728 728
======================================
+ Hits 2852 2853 +1
+ Misses 60 59 -1
+ Partials 44 43 -1 ☔ View full report in Codecov by Sentry. |
Pylint has flagged some issues after some changes in 2fd4dae. This PR could be used to correct these.
|
Whoops, that is completely my fault. Thank you for flagging and catching that, @purva-thakre ! |
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.
Minor comments, but LGTM! MWR! 🚀
Description
Fixes #245
Fixes #238
Todos
Notable points that this PR has either accomplished or will accomplish.
Questions
Status