Skip to content

Trim extra displayed carriage returns in diffs and files with CRLFs

Stan Hu requested to merge fix-extra-new-line-crlf into master

What does this MR do?

This MR strips out extra carriage returns at the end of each tokenized string.

Are there points in the code the reviewer needs to double check?

Does this remove more than we need?

Why was this MR needed?

Windows users would see extra lines in their files. Newline characters are already added with every line in the HTML formatter.

What are the relevant issue numbers?

#20440 (closed), gitlab-com/support-forum#892, #20468 (closed)

Does this MR meet the acceptance criteria?

Merge request reports