-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Fixed docstring of return value of the srloop and srploop functions #4060
base: master
Are you sure you want to change the base?
Conversation
Thanks for this PR! |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #4060 +/- ##
=======================================
Coverage 81.95% 81.95%
=======================================
Files 327 327
Lines 75540 75554 +14
=======================================
+ Hits 61911 61923 +12
- Misses 13629 13631 +2
|
Retrigger tests |
…nly on conf.iface
Added another fix in the docstring, regarding the AsyncSniffer/sniff docstring. |
So I doubt this is very useful. This is basically only ever shown in help(), in which case I would much rather have a user friendly typing. Generally, the current docstrings is also very unhelpful so I believe it would be better to simply redo it using words, and sphinx |
Fixed return value of the srloop and srploop functions. The return value was updated in the comment, but not in the docstring.