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

Add runtime check for valid Python interpreter #7365

Merged
merged 13 commits into from Mar 14, 2019

Remove unneeded `as e` in with statement

Sorry for the email spam. This is ready for review now.
  • Loading branch information...
Eric-Arellano committed Mar 12, 2019
commit 081f74670d21c7114a1a85070795fe12220f6c74
@@ -29,5 +29,5 @@ def test_ensure_valid_interpreter(self):
self.assertIn("unsupported Python interpreter", str(e))
self.assertIn(current_interpreter_version, str(e))
self.assertIn(bypass_env, str(e))
with mock.patch.dict(os.environ, {bypass_env: "1"}) as e:
with mock.patch.dict(os.environ, {bypass_env: "1"}):
PantsLoader.ensure_valid_interpreter()
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.