Skip to content

Commit 8278a42

Browse files
authored
Merge pull request #88 from PierreGode/PierreGode-patch-4
Update review.yml
2 parents f525322 + 9919858 commit 8278a42

File tree

1 file changed

+10
-2
lines changed

1 file changed

+10
-2
lines changed

.github/workflows/review.yml

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,13 @@ jobs:
3030
import requests
3131
import json
3232
33+
# Helper function to extract valid line numbers
34+
def extract_line_number(line_info):
35+
try:
36+
return int(line_info.split(" ")[1]) # Extract integer after "Line"
37+
except (IndexError, ValueError):
38+
return None # Return None if conversion fails
39+
3340
# Gather GitHub event details
3441
event_path = os.environ.get('GITHUB_EVENT_PATH')
3542
with open(event_path, 'r') as f:
@@ -79,8 +86,9 @@ jobs:
7986
line_info = parts[0].strip()
8087
description = ":".join(parts[1:]).strip()
8188
82-
if "Line" in line_info:
83-
line_number = int(line_info.split(" ")[1]) # Extract line number
89+
# Extract valid line number
90+
line_number = extract_line_number(line_info)
91+
if line_number is not None:
8492
inline_comments.append({
8593
"path": filename,
8694
"line": line_number,

0 commit comments

Comments
 (0)