sharwell
commented
8 years ago @DavidRogers I have several additional pull requests ready to submit with proposed improvements, but all of them would have merge conflicts with this pull request. The strategy proposed by this pull request is helpful especially when multiple contributors are involved. Can you take a look at the approach I used as well (with setting you as the author of commit e27b399) and make a decision regarding whether to accept this or not?
In the meantime, I'll create issues so you are aware of the specifics of the other pull requests I'm interested in submitting.
Thanks! :smile:
DavidRogers
This commit updates the repository to properly normalize line endings for text files in standard git fashion. The net impact is the new setting added to .gitattributes is now applied uniformly, so contributors that have their local environment configured with settings that differ from one another do not end up checking out and committing text files with differing (or worse mixed) line endings.
:memo: I intentionally set the author of commit e27b399 (where the new settings were applied to all files) to @DavidRogers, so as to prevent "taking ownership" of the entire codebase from the perspective of
git blame.:bulb: The specific commands used to normalize the line endings after creation of the .gitattributes file were the following. After these commands, I committed all changed files (with the exception of files removed in #17, to reduce the merge conflict to just .gitignore).
@DavidRogers There is a minor conflict between this pull request and #17. I will gladly update either one to eliminate the merge conflict if you would like.