Skip to content

Commit c37e728

Browse files
committedApr 11, 2023
rename maybe_delink to maybe_remove_mention
1 parent f470c29 commit c37e728

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed
 

Diff for: ‎src/tools/publish_toolstate.py

+3-3
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ def gh_url():
8686
return os.environ['TOOLSTATE_ISSUES_API_URL']
8787

8888

89-
def maybe_unlink(message):
89+
def maybe_remove_mention(message):
9090
# type: (str) -> str
9191
if os.environ.get('TOOLSTATE_SKIP_MENTIONS') is not None:
9292
return message.replace("@", "")
@@ -109,7 +109,7 @@ def issue(
109109
else:
110110
status_description = 'no longer builds'
111111
request = json.dumps({
112-
'body': maybe_unlink(textwrap.dedent('''\
112+
'body': maybe_remove_mention(textwrap.dedent('''\
113113
Hello, this is your friendly neighborhood mergebot.
114114
After merging PR {}, I observed that the tool {} {}.
115115
A follow-up PR to the repository {} is needed to fix the fallout.
@@ -285,7 +285,7 @@ def update_latest(
285285
issue_url = gh_url() + '/{}/comments'.format(number)
286286
response = urllib2.urlopen(urllib2.Request(
287287
issue_url,
288-
json.dumps({'body': maybe_unlink(message)}).encode(),
288+
json.dumps({'body': maybe_remove_mention(message)}).encode(),
289289
{
290290
'Authorization': 'token ' + github_token,
291291
'Content-Type': 'application/json',

0 commit comments

Comments
 (0)
Please sign in to comment.