-
-
Notifications
You must be signed in to change notification settings - Fork 414
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
Question: Can no-cover
"cover" the rest of the function?
#1741
Comments
Currently there is not a pragma comment that means "ignore the rest of this." The simplest approach that you haven't mentioned is to put that tail of the function into a new function, then you can pragma the call and the function definition with one comment each. It will still look a bit odd though. |
Oh yeah - something like ... ofc that will still need the I can go with that. Thank you for the idea 🙃 |
Is your feature request related to a problem? Please describe.
I have the end of the
main()
function like so:I'd like to ignore it with a
no-cover
.Basically, the rest of the function would be hard to hit (somehow argparse does not "match" command with its function, or dev forgets to
return cmd_handler()
.I'd like to avoid the
if return else
pattern since, on top of that, I agree with the https://pylint.pycqa.org/en/latest/user_guide/messages/refactor/no-else-return.html check aestheticallyAny solutions I missed?
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context about the feature request here.
The text was updated successfully, but these errors were encountered: