Skip to content

Update .gitattributes for new file types#504

Merged
leerho merged 2 commits into
masterfrom
tisonkun-patch-2
Jun 19, 2026
Merged

Update .gitattributes for new file types#504
leerho merged 2 commits into
masterfrom
tisonkun-patch-2

Conversation

@tisonkun

Copy link
Copy Markdown
Member

See apache/datasketches-rust#92 (comment).

I don't know if it works but it should at least do no harm.

See apache/datasketches-rust#92 (comment).

I don't know if it works but it should at least do no harm.

Copilot AI left a comment

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.

Pull request overview

Adds a repository-wide .gitattributes to standardize line-ending behavior (primarily enforcing LF for text files) and to mark certain file types as binary/export-ignored, aligning with cross-platform development expectations referenced in the linked discussion.

Changes:

  • Introduces .gitattributes with text=auto plus enforced eol=lf for text files.
  • Adds CRLF checkout rules for Windows batch scripts (*.bat, *.cmd).
  • Marks common image formats as binary and excludes selected metadata directories/files from git archive exports.

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

Comment thread .gitattributes Outdated
Comment thread .gitattributes Outdated
Comment thread .gitattributes Outdated
@coveralls

coveralls commented Jun 18, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 27806766820

Coverage remained the same at 82.199%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 21476
Covered Lines: 17653
Line Coverage: 82.2%
Coverage Strength: 1354330.69 hits per line

💛 - Coveralls

@proost proost left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you!

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@leerho leerho merged commit 0bab259 into master Jun 19, 2026
31 checks passed
@leerho leerho deleted the tisonkun-patch-2 branch June 19, 2026 18:09
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.

5 participants