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
Fix api doc params #629
Fix api doc params #629
Conversation
Oh I see the history is messy, it has a merge commit that I didn't want to push. I hope you can squash on merge. |
All good, thank you very much for the PR, I'm cleaning it up! |
Add minimal docstrings to public methods so ReadTheDocs will display them. Add autodoc mock import for lief so RTD can generate HTML for tools. This fixes issue #626
This pull request introduces 1 alert and fixes 12 when merging ba783f6 into d3db7fe - view on LGTM.com new alerts:
fixed alerts:
|
Failed CI bcos I sent the wrong version. Hang on. |
I don't want you to waste your time cleaning messes I create, I can clean them myself, just comment on what I miss, thanks. |
Codecov Report
@@ Coverage Diff @@
## main #629 +/- ##
=======================================
Coverage 37.06% 37.06%
=======================================
Files 39 39
Lines 7724 7724
=======================================
Hits 2863 2863
Misses 4861 4861
Continue to review full report at Codecov.
|
Excellent, thank you for the fix, you did great work there! |
Repairs #626