Commit 0b8fcc55 by stv

Fix PEP8: E713 test for membership

should be 'not in'
parent cdf7876a
...@@ -98,7 +98,7 @@ def ensure_pr_fetch(): ...@@ -98,7 +98,7 @@ def ensure_pr_fetch():
""" """
modified = False modified = False
remotes = git.remote().splitlines() remotes = git.remote().splitlines()
if not "edx" in remotes: if 'edx' not in remotes:
git.remote("add", "edx", "https://github.com/edx/edx-platform.git") git.remote("add", "edx", "https://github.com/edx/edx-platform.git")
modified = True modified = True
# it would be nice to use the git-python API to do this, but it doesn't seem # it would be nice to use the git-python API to do this, but it doesn't seem
...@@ -251,9 +251,9 @@ def ensure_github_creds(attempts=3): ...@@ -251,9 +251,9 @@ def ensure_github_creds(attempts=3):
else: else:
config = {} config = {}
# update config # update config
if not "credentials" in config: if 'credentials' not in config:
config["credentials"] = {} config["credentials"] = {}
if not "api.github.com" in config["credentials"]: if 'api.github.com' not in config['credentials']:
config["credentials"]["api.github.com"] = {} config["credentials"]["api.github.com"] = {}
config["credentials"]["api.github.com"]["username"] = username config["credentials"]["api.github.com"]["username"] = username
config["credentials"]["api.github.com"]["token"] = token config["credentials"]["api.github.com"]["token"] = token
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment