Not all leading tab characters in diagnostic messages are replaced with spaces
#41 268 ouverte le 27 oct. 2020
Métriques du dépôt
- Stars
- (48 455 stars)
- Métriques de merge PR
- (Merge moyen 6j 17h) (9 PRs mergées en 30 j)
Description
The error diagnostics that are used by default do not replace ALL leading tab characters with a single space.
As one would expect from seeing this code line in souce code: https://github.com/microsoft/TypeScript/blob/85ef91e2233fa384e980005c109fadc04376ce3e/src/compiler/program.ts#L410
This line only replaces the first leading tab character. Which causes way too much leading whitespace printed in the logs when errors occur during tsc build.
TypeScript Version: 4.0.3
Search Terms: tabs diagnostics, reporter, formatCodeSpan, formatDiagnosticsWithColorAndContext, leading tabs,
Expected behavior: Source code has four leading tabs for indentation.
[line number] [gutter seperator] [space] [space] [space] [space] [line content]
Actual behavior:
[line number] [gutter seperator] [space] [tab] [tab] [tab] [line content]
PR
I can submit a PR for this that would change this line:
lineContent = lineContent.replace("\t", " "); // convert tabs to single spaces
to:
lineContent = lineContent.replace(/\t/gy, " "); // convert all leading tabs to single spaces