-
Notifications
You must be signed in to change notification settings - Fork 326
Description
The test failures stem from #5780 being based on an older revision — it didn't account for the changes introduced in #5768.
Args:
comments: List of comment objects (sorted by date)
users_info: Dictionary mapping PHIDs to user info with trust status
Returns:
Tuple of (sanitized_comments, filtered_count)
"""
from copy import copy
# Walk backwards to find last trusted comment (from MOCO)
last_trusted_index = -1
for i in range(len(comments) - 1, -1, -1):
comment_is_trusted = users_info.get(comments[i].author_phid, {}).get(
"is_trusted", False
)
if comment_is_trusted:
last_trusted_index = i
break
# Process comments and apply filtering
filtered_count = 0
sanitized_comments = []
for i, comment in enumerate(comments):
author = users_info.get(comment.author_phid)
comment_is_trusted = author and (
> author["is_trusted"] or author["is_trusted_bot"]
^^^^^^^^^^^^^^^^^^^^^^^^
)
E KeyError: 'is_trusted_bot'
/bugbug/bugbug/tools/core/platforms/phabricator.py:180: KeyError
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels