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

Misc cleanup of all text files #7069

Closed
wants to merge 1 commit into from

Conversation

basicfeatures
Copy link

  • Remove CR+LF (^M)
  • Remove trailing whitespace
  • Compress two or more consecutive blank lines
  • Ensure newline at EOF

https://unix.stackexchange.com/questions/18743/whats-the-point-in-adding-a-new-line-to-the-end-of-a-file

- Remove CR+LF (^M)
- Remove trailing whitespace
- Compress two or more consecutive blank lines
- Ensure newline at EOF

https://unix.stackexchange.com/questions/18743/whats-the-point-in-adding-a-new-line-to-the-end-of-a-file
@matzbot matzbot requested a review from a team January 5, 2023 16:06
@hsbt
Copy link
Member

hsbt commented Jan 6, 2023

We don't intend to do these trivial changes with large pull-request. Please separate these changes and submit each upstreams.

@hsbt hsbt closed this Jan 6, 2023
matzbot pushed a commit that referenced this pull request Jan 6, 2023
@nobu
Copy link
Member

nobu commented Jan 6, 2023

  • Remove CR+LF (^M)

"*.bat" files need CR. Don't remove.

  • Compress two or more consecutive blank lines

Often intentionally placing blank lines to separate blocks.

  • Ensure newline at EOF

Added to files under "enc/trans/*-tbl.rb" at dd0b401.
I couldn't confirm other files because this PR is too huge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants