Skip to content

Commit 1c4e3fa

Browse files
aristathclaude
andcommitted
Fix regex group numbers after adding Methods capture
The regex now captures Methods percentage in group 2, which shifted all subsequent group numbers: - Group 1: Class name - Group 2: Methods percentage (captured but not used) - Group 3: Lines percentage - Group 4: Covered lines count - Group 5: Total lines count Updated the Python code to use the correct group numbers (3, 4, 5) instead of the old numbers (2, 3, 4). This fixes the ValueError: invalid literal for int() with base 10: '91.92' 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 1e6e976 commit 1c4e3fa

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

.github/workflows/code-coverage.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -203,9 +203,10 @@ jobs:
203203
match = re.search(r'^([\w\\]+)\s+Methods:\s+([\d.]+)%.*Lines:\s+([\d.]+)%\s+\(\s*(\d+)/\s*(\d+)\)', line)
204204
if match:
205205
class_name = match.group(1)
206-
line_percent = float(match.group(2))
207-
covered_lines = int(match.group(3))
208-
total_lines = int(match.group(4))
206+
# Group 2 is methods percentage (not used)
207+
line_percent = float(match.group(3)) # Lines percentage
208+
covered_lines = int(match.group(4)) # Covered lines count
209+
total_lines = int(match.group(5)) # Total lines count
209210
return class_name, line_percent, covered_lines, total_lines
210211
return None, None, None, None
211212

0 commit comments

Comments
 (0)