-
Notifications
You must be signed in to change notification settings - Fork 0
Added documentation, tests, and more #21
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
… quotes consistently. Added developer info to README.md
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #21 +/- ##
==========================================
+ Coverage 95.07% 97.87% +2.80%
==========================================
Files 3 3
Lines 284 283 -1
==========================================
+ Hits 270 277 +7
+ Misses 14 6 -8 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
kwabenantim
left a comment
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.
Everything looks good to me, just needs a minor update to flake8 settings.
Co-authored-by: Kwabena Amponsah <kwabenantim@yahoo.com>
Fixes #14
Also raised some questions in #15 #16 #17 #18 and there are still other docstrings in
Tracethat could use a bit of work (what exactly is the output, what is a "trace file index" and what is the format "sweeps" are returned in etc