Skip to content

Commit 10b63ac

Browse files
authored
Make "!buildbot" command ignores the case (#409)
1 parent 0e38ba0 commit 10b63ac

File tree

2 files changed

+6
-5
lines changed

2 files changed

+6
-5
lines changed

master/custom/pr_testing.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -254,18 +254,19 @@ def handle_issue_comment(self, payload, event):
254254
)
255255
return ([], "git")
256256

257-
builder_filter_fn = re.compile(builder_filter)
257+
# This code is related to GitHubPrScheduler
258+
builder_filter_fn = re.compile(builder_filter, re.IGNORECASE)
258259
matched_builders = [
259-
builder
260-
for builder in self.builder_names
260+
builder_name
261+
for builder_name in builder_names
261262
if builder_filter_fn.search(builder)
262263
]
263264
if not matched_builders:
264265
log.msg(f"GitHub PR #{number}: regex {builder_filter!r} "
265266
f"did not match any builder", logLevel=logging.DEBUG)
266267
yield self._post_comment(
267268
payload["issue"]["comments_url"],
268-
f"The regex {builder_filter!r} did not match any buildbot builder."
269+
f"The regex {builder_filter!r} did not match any buildbot builder. "
269270
f"Is the requested builder in the list of stable builders?",
270271
)
271272
return (changes, "git")

master/custom/schedulers.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ def addBuildsetForChanges(self, **kwargs):
2222
if builder_filter and builder_names:
2323
log.msg("Found builder filter: {}".format(builder_filter))
2424
builder_filter, _ = builder_filter
25-
matcher = re.compile(builder_filter)
25+
matcher = re.compile(builder_filter, re.IGNORECASE)
2626
builder_names = [
2727
builder_name
2828
for builder_name in builder_names

0 commit comments

Comments
 (0)