Skip to content

chore: add text=auto catch-all to .gitattributes#1051

Merged
Aureliolo merged 2 commits intomainfrom
chore/gitattributes-text-auto
Apr 3, 2026
Merged

chore: add text=auto catch-all to .gitattributes#1051
Aureliolo merged 2 commits intomainfrom
chore/gitattributes-text-auto

Conversation

@Aureliolo
Copy link
Copy Markdown
Owner

Adds * text=auto as the first line of .gitattributes to 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 existing eol=lf/eol=crlf overrides.

Changes

  • Add * text=auto catch-all rule before the existing Go/shell/Docker/PowerShell overrides
  • Existing eol=lf and eol=crlf overrides continue to take precedence

Test plan

  • Verified git add -A && git diff --staged --stat no longer produces CRLF warnings for .md and other text files on Windows

Copilot AI review requested due to automatic review settings April 3, 2026 20:44
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 3, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 5ddf8046-2d8a-49f2-bbda-a088c8d78afd

📥 Commits

Reviewing files that changed from the base of the PR and between 4b3bc8d and e911ec8.

📒 Files selected for processing (1)
  • .gitattributes
📜 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)
  • GitHub Check: Dependency Review
🧰 Additional context used
🧠 Learnings (6)
📚 Learning: 2026-03-15T18:17:43.675Z
Learnt from: CR
Repo: Aureliolo/synthorg PR: 0
File: CLAUDE.md:0-0
Timestamp: 2026-03-15T18:17:43.675Z
Learning: Pre-commit hooks: trailing-whitespace, end-of-file-fixer, check-yaml, check-toml, check-json, check-merge-conflict, check-added-large-files, no-commit-to-branch (main), ruff check+format, gitleaks, hadolint (Dockerfile linting).

Applied to files:

  • .gitattributes
📚 Learning: 2026-04-02T08:47:46.313Z
Learnt from: CR
Repo: Aureliolo/synthorg PR: 0
File: CLAUDE.md:0-0
Timestamp: 2026-04-02T08:47:46.313Z
Learning: Pre-commit hooks enforce: trailing-whitespace, end-of-file-fixer, check-yaml, check-toml, check-json, check-merge-conflict, check-added-large-files, no-commit-to-branch (main), ruff check+format, gitleaks, hadolint, golangci-lint + go vet (CLI, conditional), no-em-dashes, no-redundant-timeout, eslint-web (web dashboard, zero warnings, conditional)

Applied to files:

  • .gitattributes
📚 Learning: 2026-04-03T20:24:03.685Z
Learnt from: CR
Repo: Aureliolo/synthorg PR: 0
File: CLAUDE.md:0-0
Timestamp: 2026-04-03T20:24:03.685Z
Learning: Pre-commit hooks enforce: trailing-whitespace, end-of-file-fixer, check-yaml, check-toml, check-json, ruff check+format, gitleaks, hadolint, golangci-lint, eslint-web.

Applied to files:

  • .gitattributes
📚 Learning: 2026-03-15T21:49:53.264Z
Learnt from: CR
Repo: Aureliolo/synthorg PR: 0
File: CLAUDE.md:0-0
Timestamp: 2026-03-15T21:49:53.264Z
Learning: Fix everything valid — never skip when review agents find valid issues (including pre-existing issues in surrounding code, suggestions, and findings adjacent to the PR's changes). No deferring, no 'out of scope' skipping.

Applied to files:

  • .gitattributes
📚 Learning: 2026-04-02T09:04:54.709Z
Learnt from: CR
Repo: Aureliolo/synthorg PR: 0
File: CLAUDE.md:0-0
Timestamp: 2026-04-02T09:04:54.709Z
Learning: Applies to cli/**/*.go : golangci-lint + go vet enforced on CLI code via pre-commit (conditional on `cli/**/*.go`) and pre-push hooks

Applied to files:

  • .gitattributes
📚 Learning: 2026-04-03T20:24:03.685Z
Learnt from: CR
Repo: Aureliolo/synthorg PR: 0
File: CLAUDE.md:0-0
Timestamp: 2026-04-03T20:24:03.685Z
Learning: Applies to cli/**/*.go : Go code: golangci-lint + go vet enforcement via pre-commit hooks and CI (conditional on `cli/**/*.go`).

Applied to files:

  • .gitattributes
🔇 Additional comments (1)
.gitattributes (1)

1-4: Good baseline line-ending policy; override precedence remains intact.

* text=auto is a solid catch-all here, and your explicit eol=lf / eol=crlf rules still correctly govern the sensitive file types below.


Walkthrough

A new catch-all rule (* text=auto) has been added to the .gitattributes file to enable Git's automatic text normalization for all repository files. This rule establishes a baseline configuration that preserves operating system-native line endings during file checkout. The change does not affect existing, more specific configuration rules for Go, template, shell, Dockerfile, and PowerShell files that already define explicit eol= directives. The modification adds four lines with no deletions.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely summarizes the main change: adding a text=auto catch-all rule to .gitattributes.
Description check ✅ Passed The description is directly related to the changeset, providing clear context about the purpose, changes made, and testing performed.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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=auto at the top of .gitattributes to enable automatic text normalization.
  • Keep existing eol=lf (Go/shell/Docker) and eol=crlf (PowerShell) overrides as-is.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

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
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

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.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 3, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA e911ec8.
Ensure 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 Files

None

coderabbitai[bot]
coderabbitai bot previously approved these changes Apr 3, 2026
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Aurelio <19254254+Aureliolo@users.noreply.github.com>
@Aureliolo Aureliolo merged commit fc65d72 into main Apr 3, 2026
24 checks passed
@Aureliolo Aureliolo deleted the chore/gitattributes-text-auto branch April 3, 2026 20:54
Aureliolo added a commit that referenced this pull request Apr 4, 2026
🤖 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>
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