Run flake8 on the whole codebase and fix violations. Once clean, the CI lint workflow (sibling issue) can be flipped from advisory to blocking.
Mixed-style hotspots noted during review: squareIdx, fileIdx, isValidFENboard, is_FEN, versionNumber — camelCase locals and methods alongside snake_case ones. PEP 8 favors snake_case for functions and variables.
Consider also running black for autoformatting once the project is single-file-split (see #38), since black produces noisy diffs on large files.
Related to #1.
Run
flake8on the whole codebase and fix violations. Once clean, the CI lint workflow (sibling issue) can be flipped from advisory to blocking.Mixed-style hotspots noted during review:
squareIdx,fileIdx,isValidFENboard,is_FEN,versionNumber— camelCase locals and methods alongside snake_case ones. PEP 8 favors snake_case for functions and variables.Consider also running
blackfor autoformatting once the project is single-file-split (see #38), sinceblackproduces noisy diffs on large files.Related to #1.