Skip to content

chore: add .gitattributes for cross-platform CRLF handling - #82

Merged
getappz merged 1 commit into
masterfrom
fix/crlf-cargo-version
Jul 7, 2026
Merged

getappz merged 1 commit into
masterfrom
fix/crlf-cargo-version

Conversation

@getappz

@getappz getappz commented Jul 7, 2026 •

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Chores
    • Added repository line-ending rules to keep text files consistent across platforms.
    • Ensured shell scripts and Rust-related files use LF endings.
    • Marked lock files as binary to avoid unwanted normalization.

- Enforce LF for Rust source, Cargo.toml, shell scripts
- Binary lock files
- Closes ponytail PR audit ticket #69
@coderabbitai

coderabbitai Bot commented Jul 7, 2026 •

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: b0db007b-ef22-4385-bea8-c9dd7a697ec8

📥 Commits

Reviewing files that changed from the base of the PR and between 7af46db and f2f6de3.

📒 Files selected for processing (1)
  • .gitattributes

📝 Walkthrough

Walkthrough

A new .gitattributes file is added to configure repository-wide text handling: auto-detection of text files, enforced LF line endings for shell scripts and Rust source files, and binary treatment for lock files.

Changes

Line-ending normalization

Layer / File(s) Summary
Add .gitattributes config
.gitattributes
Adds text auto-detection, forces LF for *.sh and Rust files (*.rs, Cargo.toml, Cargo.lock), and marks *.lock files as binary.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related issues

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is missing the required Summary, Test plan, and reviewer notes sections. Add the template sections with a brief summary, the cargo test/clippy/fmt plan, and notes on risk areas and backward compatibility.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: adding .gitattributes to control line endings.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/crlf-cargo-version

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

@getappz
getappz merged commit 705491e into master Jul 7, 2026
8 checks passed
@getappz
getappz deleted the fix/crlf-cargo-version branch July 7, 2026 17:36
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 7, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant