Skip to content

Run repo-wide pre-commit cleanup#102

Merged
Muhtasham merged 1 commit intoCodeClash-ai:mainfrom
Muhtasham:feat/ruff-cleanup
May 5, 2026
Merged

Run repo-wide pre-commit cleanup#102
Muhtasham merged 1 commit intoCodeClash-ai:mainfrom
Muhtasham:feat/ruff-cleanup

Conversation

@Muhtasham
Copy link
Copy Markdown
Contributor

Summary

  • Fix current Ruff lint drift in BattleCode23 and Chess without changing arena behavior.
  • Run the existing pre-commit hooks across the repo, keeping the resulting mechanical formatting / EOF cleanup.
  • Pin the typos hook to a concrete release, allowlist the intentional mis variable name, fix one real typo, and add CI coverage for pre-commit run --all-files.

Verification

  • uv run pre-commit run --all-files
  • uv run ruff check .
  • uv run ruff format . --check
  • uv run pytest tests/arenas/test_battlecode23.py tests/arenas/test_chess.py
  • uv run pytest -v --cov --cov-branch --cov-report=xml -n auto (185 passed)
  • uv run mkdocs build
  • git diff --check

Opened as draft intentionally so this can run CI and be reviewed separately from the arena PRs.

@Muhtasham Muhtasham marked this pull request as ready for review May 5, 2026 09:41
@Muhtasham Muhtasham merged commit 7fcc206 into CodeClash-ai:main May 5, 2026
3 checks passed
@Muhtasham Muhtasham deleted the feat/ruff-cleanup branch May 5, 2026 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant