Skip to content

Commit

Permalink
Remove the 'join' for blacklist item
Browse files Browse the repository at this point in the history
We no longer blacklist multiple items at once, so we don't need the joins here.
  • Loading branch information
teward committed Jan 17, 2017
1 parent 5dfba55 commit 1511719
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions gitmanager.py
Expand Up @@ -52,7 +52,7 @@ def add_to_blacklist(**kwargs):
last_character = blacklist_file.read()[-1:]
if last_character != "\n":
blacklist_file.write("\n")
blacklist_file.write("\n".join(item_to_blacklist) + "\n")
blacklist_file.write(item_to_blacklist + "\n")

# Checkout a new branch (for PRs for non-code-privileged people)
branch = "auto-blacklist-{0}".format(str(time.time()))
Expand All @@ -62,7 +62,7 @@ def add_to_blacklist(**kwargs):
git.reset("HEAD")

git.add(blacklist_file_name)
git.commit("-m", u"Auto blacklist of {0} by {1} --autopull".format(", ".join(item_to_blacklist), username))
git.commit("-m", u"Auto blacklist of {0} by {1} --autopull".format(item_to_blacklist, username))

if code_permissions:
git.checkout("master")
Expand Down

0 comments on commit 1511719

Please sign in to comment.