Skip to content

Commit 1f9bb06

Browse files
committed
Merge in PR ufs-community#3132
1 parent fc161be commit 1f9bb06

File tree

1 file changed

+10
-4
lines changed

1 file changed

+10
-4
lines changed

.github/scripts/check_log_warnings_remarks.py

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,10 @@ def call_API(self, endpoint):
3434

3535
api_call = APICall(endpoint)
3636
response = requests.get(api_call.url, headers=api_call.header)
37+
if response.status_code != 200:
38+
logging.warning(response)
39+
print(response)
40+
sys.exit(1)
3741
response = json.loads(response.text)
3842

3943
return response
@@ -96,15 +100,15 @@ def _get_pr_data(self, commit):
96100
def compare_results(self, pr_log, base_log):
97101
"""Compare warnings/remarks for PR head and base commits to determine whether warnings/remarks have increased."""
98102

99-
increases = {'warnings': [], 'remarks': []}
103+
increases = {'warnings': {}, 'remarks': {}}
100104

101105
for test in pr_log:
102106
# Check warnings
103107
if pr_log[test][0] > base_log[test][0]:
104-
increases['warnings'].append(test)
108+
increases['warnings'].update({test: pr_log[test][0] - base_log[test][0]})
105109
# Check remarks
106110
if pr_log[test][1] > base_log[test][1]:
107-
increases['remarks'].append(test)
111+
increases['remarks'].update({test: pr_log[test][1] - base_log[test][1]})
108112

109113
return increases
110114

@@ -118,7 +122,9 @@ def print_html_results(dict):
118122
for category in results.keys():
119123
if results[category]:
120124
mdFile.write(f"\n<h3>{machine.upper()}</h3>\n")
121-
unordered_list = [f"**{category.title()}:**", dict[machine][category]]
125+
unordered_list = [f"**{category.title()}:**"]
126+
for test, value in dict[machine][category].items():
127+
unordered_list.append(f"{test}: {value}")
122128
mdFile.new_list(unordered_list, marked_with='*')
123129
return mdFile.get_md_text()
124130

0 commit comments

Comments
 (0)