Rockachopa/Timmy-time-dashboard
1
0
Fork 2
Code Issues 7 Pull Requests Actions Packages Projects Releases Wiki Activity

[claude] Fix ruff S105/S106/B017/E402 errors in bannerlord (#1161) #1165

Merged
Timmy merged 1 commits from :claude/issue-1161 into main 2026-03-23 19:56:08 +00:00
Conversation 0 Commits 1 Files Changed 4 +82 -39

1 Commits

Author SHA1 Message Date
Alexander Whitestone
794f663fc4 fix: suppress ruff S105/S106/B017/E402 false positives in bannerlord
Some checks failed
Tests / lint (pull_request) Has been cancelled
Details
Tests / test (pull_request) Has been cancelled
Details 
- Add # noqa: S106 to KingSubgoal(token="RECRUIT") default fallback in king.py
- Add # noqa: S105 to all subgoal.token string comparisons in vassals.py
  (EXPAND_TERRITORY, RECRUIT, TRAIN, FORTIFY, TRADE, ALLY, SPY)
- Change pytest.raises(Exception) to pytest.raises(ValueError) in test_models.py
  to fix B017 blind exception assertion
- Move pytestmark after local import in test_metabolic_router.py to fix E402

tox -e format now passes cleanly.

Fixes #1161
 2026-03-23 15:52:07 -04:00