nordic-dev.net/rust
nordic-dev.net/rust
2
0
Fork 0
mirror of https://github.com/rust-lang/rust.git synced 2024-11-27 01:04:03 +00:00
Code Issues Packages Projects Releases Wiki Activity

Rollup merge of #69449 - JohnTitor:toolstate-ping, r=Mark-Simulacrum

Browse Source 
Do not ping PR reviewers in toolstate breakage

As per https://github.com/rust-lang/rust/issues/69419#issuecomment-590470520

We should also remove author part?
This commit is contained in:
Dylan DPC 2020-02-28 01:55:40 +01:00 committed by GitHub
commit ab45408a4c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/tools/publish_toolstate.py
View File

@ -131,7 +131,6 @@ def issue(
assignees,
relevant_pr_number,
relevant_pr_user,
pr_reviewer,
):
# Open an issue about the toolstate failure.
if status == 'test-fail':
@ -147,11 +146,11 @@ def issue(
cc @{}, do you think you would have time to do the follow-up work?
If so, that would be great!
cc @{}, the PR reviewer, and nominating for compiler team prioritization.
And nominating for compiler team prioritization.
''').format(
relevant_pr_number, tool, status_description,
REPOS.get(tool), relevant_pr_user, pr_reviewer
REPOS.get(tool), relevant_pr_user
)),
'title': '`{}` no longer builds after {}'.format(tool, relevant_pr_number),
'assignees': list(assignees),
@ -236,7 +235,7 @@ def update_latest(
try:
issue(
tool, create_issue_for_status, MAINTAINERS.get(tool, ''),
relevant_pr_number, relevant_pr_user, pr_reviewer,
relevant_pr_number, relevant_pr_user,
)
except urllib2.HTTPError as e:
# network errors will simply end up not creating an issue, but that's better