Skip to content

Enforce LF line endings: .gitattributes + ruff format settings - #17

Merged
ctoth merged 1 commit into
masterfrom
chore/enforce-lf-endings
Jul 27, 2026
Merged

Enforce LF line endings: .gitattributes + ruff format settings#17
ctoth merged 1 commit into
masterfrom
chore/enforce-lf-endings

Conversation

@ctoth

@ctoth ctoth commented Jul 27, 2026

Copy link
Copy Markdown
Owner

Adds line-ending enforcement the repo previously lacked (no .gitattributes, no ruff config, core.autocrlf=false), which had let four Python files drift to mixed/CRLF endings against the repo's 123-file LF convention.

  • .gitattributes: * text=auto eol=lf; *.are -text so the MUD area fixtures (test/rom/*.are) stay byte-exact — they are test data and must never be renormalized.
  • pyproject.toml: [tool.ruff] with [tool.ruff.format] line-ending = "lf".
  • Renormalized the four offenders to LF: area_reader/__init__.py, area_reader/constants.py, conftest.py, test_area_reader.py.

Verification: after the change git ls-files --eol shows every tracked text file as i/lf, with only the two protected .are fixtures non-text; full suite 940 passed in 69.55s (identical count to master — no content change).

Note on merge order: #16 touches constants.py and test_area_reader.py with the current (pre-normalization) endings. Merge #16 first, then I'll rebase this branch over master so the renormalization covers its additions; merging this first will make #16 conflict on every line of those files.

🤖 Generated with Claude Code

https://claude.ai/code/session_01F8kWxXQUY73XMXyeUYhkLd

The repo was 123 files LF with 4 Python files mixed/CRLF and nothing
enforcing endings. Add .gitattributes (* text=auto eol=lf, .are fixtures
protected as binary), ruff format line-ending=lf, and renormalize the
four offenders to LF.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01F8kWxXQUY73XMXyeUYhkLd
@ctoth
ctoth force-pushed the chore/enforce-lf-endings branch from cd94f36 to 4acc3ea Compare July 27, 2026 06:39
@ctoth
ctoth merged commit 24f9b64 into master Jul 27, 2026
4 checks passed
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