Skip to content

fix(biome): stabilize LF line endings with .gitattributes - #5

Merged
greythx merged 1 commit into
mainfrom
fix/biome-lf-line-endings
Aug 12, 2026
Merged

fix(biome): stabilize LF line endings with .gitattributes#5
greythx merged 1 commit into
mainfrom
fix/biome-lf-line-endings

Conversation

@greythx

@greythx greythx commented Aug 12, 2026

Copy link
Copy Markdown
Owner

Fixes the post-merge lint regression.

Problem: After the formatting PR merged, core.autocrlf=true rewrote working-tree files to CRLF on checkout, but biome.json mandates lineEnding: "lf", so bun run lint failed again on 12 files.

Fix: Commit a .gitattributes (* text=auto eol=lf) so every checkout/clone yields LF regardless of autocrlf — verified with a fresh local clone (0 CRLF). Full verification green: lint, format, build, 54 tests.

Copilot AI lite review requested due to automatic review settings August 12, 2026 21:18
@vercel

vercel Bot commented Aug 12, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
prophetly Building Building Preview Aug 12, 2026 9:18pm

@greythx
greythx merged commit a32626e into main Aug 12, 2026
1 of 2 checks passed
@greythx
greythx deleted the fix/biome-lf-line-endings branch August 12, 2026 21:18

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds repository-level Git line-ending normalization to prevent Biome lint/format failures caused by CRLF checkouts when core.autocrlf=true.

Changes:

  • Introduces a .gitattributes rule to enforce LF (* text=auto eol=lf) across checkouts/clones.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .gitattributes
@@ -0,0 +1 @@
* text=auto eol=lf No newline at end of file
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.

2 participants