Skip to content

Fix extra \r added to newlines on windows for old_stream#58

Merged
MHendricks merged 3 commits intomainfrom
mikeh/newline_windows_fix
Mar 13, 2026
Merged

Fix extra \r added to newlines on windows for old_stream#58
MHendricks merged 3 commits intomainfrom
mikeh/newline_windows_fix

Conversation

@MHendricks
Copy link
Copy Markdown
Member

The newer io.TextIOWrapper implementation was leading to \r\n having \n replaced with \r\n on windows leading to \r\r\n newlines. This is not visible in PrEditor's console but in stdout/err.

Checklist

  • I have read the CONTRIBUTING.md document
  • I formatted my changes with black
  • I linted my changes with flake8
  • I have added documentation regarding my changes where necessary
  • Any pre-existing tests continue to pass
  • Additional tests were made covering my changes

Types of Changes

  • Bugfix (change that fixes an issue)
  • New Feature (change that adds functionality)
  • Documentation Update (if none of the other choices apply)

@MHendricks MHendricks force-pushed the mikeh/newline_windows_fix branch from 8df95aa to 9996848 Compare March 11, 2026 19:53
The newer `io.TextIOWrapper` implementation was leading to `\r\n` having
`\n` replaced with `\r\n` on windows leading to `\r\r\n` newlines. This
is not visible in PrEditor's console but in stdout/err.
@MHendricks MHendricks force-pushed the mikeh/newline_windows_fix branch from 9996848 to 48eb2c2 Compare March 11, 2026 20:05
@MHendricks MHendricks force-pushed the mikeh/newline_windows_fix branch from 49a9c83 to 4e5e42f Compare March 11, 2026 21:19
@MHendricks MHendricks merged commit 9ecfb9f into main Mar 13, 2026
37 checks passed
@MHendricks MHendricks deleted the mikeh/newline_windows_fix branch March 13, 2026 23:14
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.

1 participant