Skip to content

Commit

Permalink
Strip '@' symbols when merging pull requests.
Browse files Browse the repository at this point in the history
Currently all of the commits with 'X' in them cause person X to
receive e-mails every time someone makes a public fork of Spark.

marmbrus who requested this.

Author: Patrick Wendell <pwendell@gmail.com>

Closes apache#1239 from pwendell/strip and squashes the following commits:

22e5a97 [Patrick Wendell] Strip '@' symbols when merging pull requests.
  • Loading branch information
pwendell authored and conviva-zz committed Sep 4, 2014
1 parent 1ed94d1 commit 664a4f7
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion dev/merge_spark_pr.py
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,9 @@ def merge_pr(pr_num, target_ref):

merge_message_flags += ["-m", title]
if body != None:
merge_message_flags += ["-m", body]
# We remove @ symbols from the body to avoid triggering e-mails
# to people every time someone creates a public fork of Spark.
merge_message_flags += ["-m", body.replace("@", "")]

authors = "\n".join(["Author: %s" % a for a in distinct_authors])

Expand Down

0 comments on commit 664a4f7

Please sign in to comment.