Skip to content

Commit fae4c47

Browse files
committed
Improve documentation
dicts for reconstructing connected events are now better explained and the comments do not disruot the workflow in the run function anymore Signed-off-by: Leo Sendelbach <s8lesend@stud.uni-saarland.de>
1 parent cfaba71 commit fae4c47

1 file changed

Lines changed: 4 additions & 6 deletions

File tree

issue_processing/issue_processing.py

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -77,15 +77,10 @@ def run():
7777
issues = load(__srcdir)
7878
# 2) re-format the issues
7979
reformat_issues(issues)
80-
# create an empty dict for external connected events, meaning connected
81-
# events that connect to an issue in another repository
82-
external_connected_events = dict()
8380
# 3) merges all issue events into one list
84-
# this step returns a dict containing all connected events that can be matched to the correct issues later
81+
external_connected_events = dict()
8582
filtered_connected_events = merge_issue_events(issues, external_connected_events)
8683
# 4) re-format the eventsList of the issues
87-
# this step also reconstructs the connections previously stored
88-
# in 'external_connected_events' and 'filtered_connected_events'
8984
reformat_events(issues, filtered_connected_events, external_connected_events)
9085
# 5) update user data with Codeface database and dump username-to-name/e-mail list
9186
insert_user_data(issues, __conf, __resdir)
@@ -539,6 +534,8 @@ def merge_issue_events(issue_data, external_connected_events):
539534
issue["eventsList"] = sorted(issue["eventsList"], key=lambda k: k["created_at"])
540535

541536
# filter out connected events which cannot be perfectly matched
537+
# and populate external_connected_events dict
538+
# because this happens in place, we do not need to return the external_connected_event dict later
542539
filtered_connected_events = dict(filter(lambda item: filter_connected_events(item[0], item[1], external_connected_events), connected_events.items()))
543540

544541
# updates all the issues by the temporarily stored referenced_by events
@@ -547,6 +544,7 @@ def merge_issue_events(issue_data, external_connected_events):
547544
if issue["number"] == value["number"]:
548545
issue["eventsList"] = issue["eventsList"] + value["eventsList"]
549546

547+
# return the filtered_connected_events dict for later reconstruction
550548
return filtered_connected_events
551549

552550

0 commit comments

Comments
 (0)