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

Fix crash with NameError through decorator during collection #243

Merged
merged 2 commits into from Mar 13, 2020

Conversation

blueyed
Copy link
Owner

@blueyed blueyed commented Feb 22, 2020

Ref pytest-dev#4984.

TODO:

  • this was a quick hack, without knowing what I was doing, needs more tests likely.

ref: pytest-dev#9355

@blueyed blueyed force-pushed the fix-nameerror_with_decorator branch from c8317a7 to 94770e8 Compare March 10, 2020 16:48
@blueyed blueyed merged commit a7dfc6f into my-master Mar 13, 2020
@blueyed blueyed deleted the fix-nameerror_with_decorator branch March 13, 2020 08:33
blueyed added a commit that referenced this pull request Sep 25, 2020
NOTE: fixing/reverting of `doc = getattr(mod, "docstring", None)` was
missed in 3153740 (pytest-dev#4723).

It was added initially in
pytest-dev#2870, but never made it into a
final Python release.  Basically reverts / revisits based on changes introduced in 734c435.

Adjusts a7dfc6f (#243).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant