docs: Add missing docstrings fetch_json_db.py #3381
Conversation
terriko
changed the title
There was a problem hiding this comment.
Hey, you've got some linter errors such as...
cve_bin_tool/fetch_json_db.py:44:6: E999 IndentationError: expected an indented block after function definition on line 35
I think mostly you need your strings indented one more level so they're in line with the code below the function definition. I put one in as an example but you can probably do the rest faster than I can through the web interface.
cve_bin_tool/fetch_json_db.py
Outdated
| """ | ||
| Initialize the Fetch_JSON_DB instance. | ||
| """ |
There was a problem hiding this comment.
| """ | |
| Initialize the Fetch_JSON_DB instance. | |
| """ | |
| """ | |
| Initialize the Fetch_JSON_DB instance. | |
| """ |
|
But the contents of the docstrings look good once we get the syntax fixed! |
|
My bad. Thank you for pointing out the exact fix. Fixed it. |
Niloth-p
force-pushed
the
docs/add-function-docstrings
branch
from
705c8ad
to
b72e565
Compare
Codecov Report
@@ Coverage Diff @@
## main #3381 +/- ##
==========================================
+ Coverage 80.72% 80.93% +0.20%
==========================================
Files 750 750
Lines 11534 11534
Branches 1560 1560
==========================================
+ Hits 9311 9335 +24
+ Misses 1786 1766 -20
+ Partials 437 433 -4
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 6 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
fixes #3380