Skip to content

Commit 3d6143d

Browse files
committed
Index coverage map by tuple instead of formatting to string
1 parent f968dd1 commit 3d6143d

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

bot/code_coverage_bot/phabricator.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ def _build_coverage_map(self, annotate, coverage_record):
9191
orig_changeset = data["node"]
9292

9393
if lineno < len(coverage_record):
94-
key = "{}-{}".format(orig_changeset, orig_line)
94+
key = (orig_changeset, orig_line)
9595
coverage_map[key] = coverage_record[lineno]
9696

9797
return coverage_map
@@ -105,7 +105,7 @@ def _apply_coverage_map(self, annotate, coverage_map):
105105
# The changeset when it was introduced.
106106
orig_changeset = data["node"]
107107

108-
key = "{}-{}".format(orig_changeset, orig_line)
108+
key = (orig_changeset, orig_line)
109109
if key in coverage_map:
110110
count = coverage_map[key]
111111
if count == -1:

0 commit comments

Comments
 (0)