search

gucong3000 / gulp-reporter

Error report for: CSSLint/EditorConfig/ESLint/HTMLHint/JSCS/JSHint/PostCSS/Standard/TSLint/XO
MIT License
17 stars 1 forks source link

feature_request(unicode): disable CROSS MARK symbol from output #129 #237

Closed gucong3000 closed 6 years ago

gucong3000 commented 6 years ago

1. Summary

It would be nice, if CROSS MARK symbol will not add to console output.

2. Argumentation

Different terminal behavior.

Compare line of output for Cygwin and ConEmu:

Cygwin:

01:016 ❌ invalid newline: crlf, expected: lf (EditorConfig end_of_line https://goo.gl/te3t0s)

ConEmu:

01:016 ❌️ invalid newline: crlf, expected: lf (EditorConfig end_of_line https://goo.gl/te3t0s)

Cygwin vs ConEmu

In ConEmu output I get VARIATION SELECTOR-16 symbol in each line.

It may create some problems in debugging process, examples:

  1. User can have a different file size, if run same command in different operating systems/terminals. User may not understand the reasons.
  2. File comparison tools show differences for each line.

3. Environment

4. Example

For one of my big projects:

ConEmu:

eclint check > SashaEClintWindows.txt 2>&1

SashaEClintWindows.txt — 1515 KB. Lines with VARIATION SELECTOR-16.

Cygwin:

eclint check &> SashaEClintCygwin.txt

SashaEClintCygwin.txt — 1473 KB. Lines without VARIATION SELECTOR-16.

I get same output, but different file size because of VARIATION SELECTOR-16 symbol.

Thanks.

gucong3000 commented 6 years ago

https://github.com/gucong3000/gulp-reporter/blob/ebf7e7a45c6163f6345a48d21416ef08e500bd12/lib/formatter.js#L151-L153