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

Remove obsolete __future__ imports #3668

Merged

Conversation

Projects
None yet
3 participants
@asottile
Copy link
Member

asottile commented Jul 8, 2018

Resolves #2319

flag=generators.compiler_flag,
dont_inherit=0,
_genframe=None,
self, filename=None, mode="exec", flag=0, dont_inherit=0, _genframe=None

This comment has been minimized.

Copy link
@asottile

asottile Jul 8, 2018

Author Member

(harder to see the change here since black reformatted it, essentially this:

-    flag=generators.compiler_flag,
+    flag=0,

this is the same as the default builtin compile(...) function -- and in versions newer than python2.3 the generators flag is ignored so the two are equivalent

@coveralls

This comment has been minimized.

Copy link

coveralls commented Jul 8, 2018

Coverage Status

Coverage increased (+0.05%) to 92.649% when pulling 4ae93a7 on asottile:remove_obsolete_future_imports into d4faa40 on pytest-dev:master.

@nicoddemus nicoddemus merged commit 9ef7878 into pytest-dev:master Jul 8, 2018

1 of 2 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@asottile asottile deleted the asottile:remove_obsolete_future_imports branch Jul 8, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.