-
Notifications
You must be signed in to change notification settings - Fork 470
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
docs: add missing docstrings to data_sources/curl_source.py #3464 #3475
Conversation
Added missing docstrings to data_sources/curl_source.py
@terriko mam I have added docstring please look at it once! |
@terriko Added missing all docstrings to curl_source.py. please review it once |
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.
removed all the spaces
Codecov Report
@@ Coverage Diff @@
## main #3475 +/- ##
==========================================
- Coverage 80.34% 80.00% -0.35%
==========================================
Files 758 758
Lines 11615 11616 +1
Branches 1571 1571
==========================================
- Hits 9332 9293 -39
- Misses 1867 1904 +37
- Partials 416 419 +3
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 11 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@terriko mam I have removed all the extra spaces. Please look at it once! |
Added missing docstrings to data_sources/curl_source.py
I have explained what a function does and have it put at the top of the function in the way that Python expects docstrings to look.
Issue no #3464