Skip to content

Enforce LF line endings with .gitattributes - #64

Merged
ThoSap merged 1 commit into
mainfrom
normalize-line-endings
Sep 4, 2026
Merged

Enforce LF line endings with .gitattributes#64
ThoSap merged 1 commit into
mainfrom
normalize-line-endings

Conversation

@ThoSap

@ThoSap ThoSap commented Sep 4, 2026

Copy link
Copy Markdown
Member

Two files in #60 arrived with CRLF line endings although .editorconfig asks for LF. .editorconfig only advises the editor, so a different tool or git client can still commit CRLF. This PR adds the git-side rule.

  • * text=auto eol=lf: every text file is stored with LF and checked out with LF on every platform. Git detects binary files itself.
  • *.bat text eol=crlf: gradlew.bat keeps CRLF in the working tree, as the Gradle wrapper expects.
  • *.jar binary, *.png binary: explicit for the two binary types in the repo.
  • [*.bat] end_of_line = crlf in .editorconfig, matching the git rule.
  • Fix for the linguist-generated pattern: the jOOQ sources live under generated/src/, not src/generated/, so the old pattern matched nothing. Verified with git check-attr.

git add --renormalize . changed no other file, so main has no CRLF left after #61.

Two files in #60 were committed with CRLF although .editorconfig asks for LF.
Git now normalizes every text file to LF on commit and checks it out as LF on
every platform, so the editor setting can no longer be bypassed. gradlew.bat
keeps CRLF in the working tree. jar and png files are marked binary.

Also fix the linguist-generated pattern: the jOOQ sources live under
generated/src/, not src/generated/, so the old pattern matched nothing.
@ThoSap ThoSap self-assigned this Sep 4, 2026
@ThoSap ThoSap added the enhancement New feature or request label Sep 4, 2026
@ThoSap
ThoSap requested a review from stplasim September 4, 2026 13:32
@ThoSap
ThoSap merged commit 0e89db0 into main Sep 4, 2026
4 checks passed
@ThoSap
ThoSap deleted the normalize-line-endings branch September 4, 2026 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants