Skip to content
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

Wrapper do not copy docstring #836

Closed
rqelibari opened this issue May 14, 2021 · 0 comments · Fixed by #922
Closed

Wrapper do not copy docstring #836

rqelibari opened this issue May 14, 2021 · 0 comments · Fixed by #922
Labels
affects-0.3 Issues related to 0.3.X Mitogen releases bug Code feature that hinders desired execution outcome

Comments

@rqelibari
Copy link

mitogen.utils.with_router does not copy the wrapped functions docstring. Many libraries (e.g. click) rely on it though.

from mitogen.utils import with_router

@with_router
def some_click_command(router):
    """This is the description of the command."""
    pass

assert(some_click_command.__doc__ == "This is the description of the command.") # raises exception
@rqelibari rqelibari added affects-0.3 Issues related to 0.3.X Mitogen releases bug Code feature that hinders desired execution outcome labels May 14, 2021
moreati added a commit to moreati/mitogen that referenced this issue Apr 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects-0.3 Issues related to 0.3.X Mitogen releases bug Code feature that hinders desired execution outcome
Projects
None yet
1 participant