Support different line endings in changelog #35
Conversation
Allow specifying line endings in options Fallback to try to detect from contents Fallback to current OS line endings
|
I added the custom line endings option so that in (unusual) cases where the changelog uses multiple line endings (and you don't want the most common one), or line endings other than CR, LF, or CRLF, you have the ability to specify it yourself (I don't see this being a likely occurrence, but it was easy enough to add) |
|
I'm concerned to see you've rewritten all of the tests to serve this very specific use case. I'd prefer it the preexisting tests were not heavily modified to ensure that this is not a breaking change. |
The existing tests are all intact. I only changed the outer structure of the test file, so all existing tests could be performed using each format of line ending (primarily to ensure more full test coverage of the different line endings, rather than only covering a couple of cases, and having the potential to silently break in other cases). The only real difference being that each test name is now prefixed with the format of line endings used. |
|
Any updates on the above? I would welcome this to be a part of the official release. |
Allow specifying line endings in options
Fallback to try to detect from contents
Fallback to current OS line endings
Fixes #34