Skip to content

Fix minor bug in astropy.utils.decorators.wraps - #4016

Merged
embray merged 2 commits into
astropy:masterfrom
embray:utils/fix-wraps-name
Jul 29, 2015
Merged

embray merged 2 commits into
astropy:masterfrom
embray:utils/fix-wraps-name

Conversation

@embray

@embray embray commented Jul 29, 2015

Copy link
Copy Markdown
Member

Fixes a bug in the wraps() decorator where the wrapper function inherited the wrapped function's name even if __name__ is not included in the 'assigned' argument (which lists attributes that shoul be copied from the wrapped function to the wrapper function).

…ited the wrapped function's name even if __name__ is not included in the 'assigned' argument (which lists attributes that shoul be copied from the wrapped function to the wrapper function).
@embray

embray commented Jul 29, 2015

Copy link
Copy Markdown
Member Author

CI failed solely due to #4015. Everything else looks fine.

embray added a commit that referenced this pull request Jul 29, 2015
Fix minor bug in astropy.utils.decorators.wraps
@embray
embray merged commit 05bb6e9 into astropy:master Jul 29, 2015
@embray
embray deleted the utils/fix-wraps-name branch July 29, 2015 19:43
embray added a commit that referenced this pull request Aug 5, 2015
Fix minor bug in astropy.utils.decorators.wraps
embray added a commit that referenced this pull request Aug 7, 2015
Fix minor bug in astropy.utils.decorators.wraps
dhomeier pushed a commit to dhomeier/astropy that referenced this pull request Aug 11, 2015
embray added a commit that referenced this pull request Aug 11, 2015
Fix minor bug in astropy.utils.decorators.wraps
embray added a commit that referenced this pull request Aug 11, 2015
Fix minor bug in astropy.utils.decorators.wraps
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant