Skip to content

Normalize line endings to LF - #203

Merged
ndgrigorian merged 3 commits into
masterfrom
normalize-line-endings
Nov 9, 2026
Merged

ndgrigorian merged 3 commits into
masterfrom
normalize-line-endings

Conversation

@antonwolfy

@antonwolfy antonwolfy commented Oct 22, 2026 •

Copy link
Copy Markdown
Collaborator

Four tracked files were stored with CRLF line endings, while every other file in the repository uses LF:

  • conda-recipe/conda_build_config.yaml
  • conda-recipe-cf/conda_build_config.yaml
  • docs/make.bat
  • mkl_random/_init_helper.py

Because the repository has no .gitattributes, git keeps whatever line endings a file was committed with. When one of these files is edited with a tool that writes LF, the diff shows every line as changed.

This PR adds the same .gitattributes that dpnp uses:

* text=auto eol=lf

It stores all text files with LF and checks them out with LF, .bat files included, as in dpnp. The four files above are renormalized with git add --renormalize .. The contents don't change: git diff --ignore-cr-at-eol shows only the new .gitattributes.

Four tracked files were stored with CRLF line endings, while the rest
of the repository uses LF. Add a .gitattributes that stores text files
with LF and checks out batch files with CRLF, and renormalize the
affected files.
Use the same .gitattributes as dpnp, so that line endings are handled
the same way across the IntelPython repositories.

@jharlow-intel jharlow-intel left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks, looks good.

@ndgrigorian
ndgrigorian merged commit cdf1926 into master Nov 9, 2026
24 checks passed
@ndgrigorian
ndgrigorian deleted the normalize-line-endings branch November 9, 2026 18:25
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.

3 participants