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.

Sign up for GitHub

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

Jump to bottom

new function safer_getattr_raise #288

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

new function safer_getattr_raise #288

wants to merge 1 commit into from
+29 −1

Conversation

d-maurer
Copy link
Contributor

Fixed #287.

safer_getattr_raise is similar to safer_getattr but handles its parameter default like getattr. In particular, it raises AttributeError if the attribute lookup fails and no default has been provided.

@d-maurer
Copy link
Contributor Author

The failing coverage test should be unrelated to this change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dataflake dataflake dataflake approved these changes

@icemac icemac Awaiting requested review from icemac

@loechel loechel Awaiting requested review from loechel

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Provide a safer_getattr variant with getattr compatible signature
2 participants
@d-maurer @dataflake