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

Port to modern inspect #81

Closed
goodboy opened this issue Sep 2, 2017 · 0 comments
Closed

Port to modern inspect #81

goodboy opened this issue Sep 2, 2017 · 0 comments
Labels
deprecation

Comments

@goodboy
Copy link
Contributor

goodboy commented Sep 2, 2017

For func signature inspection in py3 we're supposed to use the new inspect.signature().
An example of how to provide dual support for py2/3 can be found here.
nicoddemus added a commit to nicoddemus/pluggy that referenced this issue Sep 12, 2017
@nicoddemus
Turned warnings into errors
1f6dc21 
- Fix two correct warnings
- Used pytest.warns() around a valid warning being issued

Should remove the warning exception for pytest-dev#81 when we tackle that.

Fix pytest-dev#78
@nicoddemus nicoddemus mentioned this issue Sep 12, 2017
Merged
@goodboy goodboy mentioned this issue Nov 23, 2017
Closed
@nicoddemus nicoddemus mentioned this issue Nov 23, 2017
Closed
RonnyPfannschmidt added a commit that referenced this issue Nov 24, 2017
@RonnyPfannschmidt
Merge pull request #94 from stephenfin/stop-using-argspec
817170d 
Fix #81: Stop using 'inspect.getargspec()'
nicoddemus added a commit to nicoddemus/pluggy that referenced this issue Nov 24, 2017
@nicoddemus
Add CHANGELOG for pytest-dev#81
d97c406
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
deprecation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@goodboy