Skip to content

Commit

Permalink
[Minor] Force line endings to be LF not CRLF (apache#2058)
Browse files Browse the repository at this point in the history
### What changes were proposed in this pull request?

Force line endings to be LF not CRLF

### Why are the changes needed?

So users using windows don't get confused and submit PRs with CRLFs in
them.

Fix: # N/A

### Does this PR introduce _any_ user-facing change?

N/A

### How was this patch tested?

I don't have a windows machine to test this on, but it's simple enough.
  • Loading branch information
justinmclean authored Feb 7, 2024
1 parent fa0ff64 commit e74e9e7
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 0 deletions.
1 change: 1 addition & 0 deletions .gitattributes
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
text eol=lf
2 changes: 2 additions & 0 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,8 @@ All files must have a license header and the build fails if any files are missin

For any bugs or new code please add unit tests to provide coverage of the code. The project may not accept code without unit tests.

All text files should use macOS/unix style line endings (LF) not windows style line endings (CRLF).

## Community and communication

Join the [community discourse group](https://gravitino.discourse.group) to discuss ideas and seek help. You are also encouraged to use GitHub discussions and follow Datastrato on social media to stay updated on project news.
Expand Down

0 comments on commit e74e9e7

Please sign in to comment.