Skip to content

.

2209377
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Merged

test(pyramid): Support alpha suffixes in version parsing #5618

.
2209377
Select commit
Loading
Failed to load commit list.
GitHub Actions / warden completed Mar 9, 2026 in 1m 49s

1 issue

Medium

Fallback code path in except block doesn't handle alpha suffixes - `tests/integrations/pyramid/test_pyramid.py:22`

The PR fixes version parsing for Python 3.8+ by using packaging.version.Version(...).release, but the fallback except ImportError block (lines 26-30 for Python < 3.8) still uses map(int, version.split('.')) which will raise ValueError on versions with alpha suffixes like '2.0a1'. This inconsistency means the fix is incomplete - tests would still fail on Python 3.6/3.7 with alpha pyramid versions.

4 skills analyzed
Skill Findings Duration Cost
code-review 1 1m 43s $0.42
find-bugs 0 49.0s $0.23
skill-scanner 0 21.2s $0.07
security-review 0 19.7s $0.06

Duration: 3m 13s · Tokens: 559.6k in / 6.6k out · Cost: $0.79 (+dedup: $0.00)