chore: add text=auto catch-all to .gitattributes#1051
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
📜 Recent review details⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
🧰 Additional context used🧠 Learnings (6)📚 Learning: 2026-03-15T18:17:43.675ZApplied to files:
📚 Learning: 2026-04-02T08:47:46.313ZApplied to files:
📚 Learning: 2026-04-03T20:24:03.685ZApplied to files:
📚 Learning: 2026-03-15T21:49:53.264ZApplied to files:
📚 Learning: 2026-04-02T09:04:54.709ZApplied to files:
📚 Learning: 2026-04-03T20:24:03.685ZApplied to files:
🔇 Additional comments (1)
WalkthroughA new catch-all rule ( 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
There was a problem hiding this comment.
Pull request overview
Adds a repo-wide .gitattributes catch-all to normalize text file line endings, primarily to reduce CRLF/LF warning noise for Windows contributors while preserving existing per-file overrides.
Changes:
- Add
* text=autoat the top of.gitattributesto enable automatic text normalization. - Keep existing
eol=lf(Go/shell/Docker) andeol=crlf(PowerShell) overrides as-is.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Code Review
This pull request updates the .gitattributes file to include a global * text=auto rule, ensuring consistent line-ending normalization across the repository. The review feedback recommends performing a repository-wide renormalization using git add --renormalize . to ensure that existing files are correctly updated in the Git index and to prevent future ghost diffs.
| @@ -1,3 +1,7 @@ | |||
| # Normalize all text files to LF in the repo, OS-native line endings on checkout. | |||
| # Silences CRLF conversion warnings on Windows. | |||
| * text=auto | |||
There was a problem hiding this comment.
When introducing a global text=auto rule to an existing repository, it is important to ensure that all currently tracked files are normalized in the Git index. Simply adding the attribute silences warnings for future operations, but existing files that were committed with CRLF line endings will remain in that state until they are touched or explicitly renormalized. To ensure a consistent state across the repository and avoid "ghost" diffs (where a small change triggers a full-file line-ending normalization in the commit), it is recommended to run git add --renormalize . and commit the changes as part of this PR or in a follow-up.
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Snapshot WarningsEnsure that dependencies are being submitted on PR branches. Re-running this action after a short time may resolve the issue. See the documentation for more information and troubleshooting advice. Scanned FilesNone |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Aurelio <19254254+Aureliolo@users.noreply.github.com>
🤖 I have created a release *beep* *boop* --- ## [0.6.1](v0.6.0...v0.6.1) (2026-04-04) ### Features * capability-aware prompt profiles for model tier adaptation ([#1047](#1047)) ([67650c5](67650c5)), closes [#805](#805) * implement procedural memory auto-generation from agent failures ([#1048](#1048)) ([55f5206](55f5206)), closes [#420](#420) * implement quality scoring Layers 2+3 -- LLM judge and human override ([#1057](#1057)) ([4a8adfe](4a8adfe)), closes [#230](#230) * token-based personality trimming via PromptProfile.max_personality_tokens ([#1059](#1059)) ([75afd52](75afd52)), closes [#1045](#1045) * workflow execution lifecycle + editor improvements ([#1058](#1058)) ([7b54262](7b54262)), closes [#1029](#1029) [#1042](#1042) ### Refactoring * **web:** address complexity and logging issues in dashboard ([#1056](#1056)) ([ada997b](ada997b)), closes [#1055](#1055) ### Documentation * comprehensive documentation refresh ([#1050](#1050)) ([c7a4259](c7a4259)) ### Tests * fix Hypothesis fuzzing infra and speed up slow unit tests ([#1044](#1044)) ([1111602](1111602)) ### Maintenance * add text=auto catch-all to .gitattributes ([#1051](#1051)) ([fc65d72](fc65d72)) * bump defu from 6.1.4 to 6.1.6 in /site ([#1062](#1062)) ([f0cc439](f0cc439)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Adds
* text=autoas the first line of.gitattributesto normalize all text files to LF in the repo and OS-native line endings on checkout.This silences the CRLF conversion warnings on Windows (e.g.
warning: LF will be replaced by CRLF the next time Git touches it) for files not already covered by the existingeol=lf/eol=crlfoverrides.Changes
* text=autocatch-all rule before the existing Go/shell/Docker/PowerShell overrideseol=lfandeol=crlfoverrides continue to take precedenceTest plan
git add -A && git diff --staged --statno longer produces CRLF warnings for.mdand other text files on Windows