Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

modified file when clonning repo #891

Closed
vncoelho opened this issue Mar 22, 2024 · 7 comments · Fixed by #892
Closed

modified file when clonning repo #891

vncoelho opened this issue Mar 22, 2024 · 7 comments · Fixed by #892

Comments

@vncoelho
Copy link
Member

image

@vncoelho
Copy link
Member Author

image

@vncoelho
Copy link
Member Author

warning: CRLF will be replaced by LF in src/ApplicationLogs/config.json.
The file will have its original line endings in your working directory

@shargon
Copy link
Member

shargon commented Mar 22, 2024

This file is with CRLF?

@cschuchardt88
Copy link
Member

@vncoelho

Run the following the commands:

git config --global core.autocrlf false # You must disable able this.

git add --renormalize .  # Update index with renormalized files
git status               # Show the files that will be normalized
git commit -m "Introduce end-of-line normalization"

@vncoelho
Copy link
Member Author

I did something similar in the PR. I will double check on Monday

@vncoelho
Copy link
Member Author

This file is with CRLF?

Not sure Shargon.

If you clone the repo and type 'git status ' you will see something different.

This was introduced in last PR.
Since everything I run is dockerized I am sure there is something to be fixed.

@cschuchardt88
Copy link
Member

@vncoelho By default git automatically changes the line endings to your computer's operating systems' line ending.

Default Line Endings per OS:

  • Windows=CRLF
  • Unix=LF
  • Linux=LF
  • MacOS=LF

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 a pull request may close this issue.

3 participants