Created by: justingrant
Currently, CRA's npm start
output only reports the line number of ESLint errors but hides the column numbers. This makes it harder for IDE users to leverage the time-saving features provided by IDEs like VSCode to deal with build errors:
- devs can't one-click to navigate from the build error message to the exact spot in the code where the problem is.
- the code editor won't show the little red sqiggly line under the exact spot in the code where the problem is.
This PR removes one line of code (originally introduced in #5174 last year) that removes column numbers from ESLint errors. Here's the current code. As you can see, it has no side effects-- it removes column numbers but doesn't do any other changes to the string. https://github.com/facebook/create-react-app/blob/4397d069d903c29927526d6fe4e7ba1b5edb136d/packages/react-dev-utils/formatWebpackMessages.js#L44-L46
Here's what existing errors look like:
./src/Calendar.tsx
Line 19: Unnecessary escape character: \. no-useless-escape
Here's what errors will look like after this PR:
./src/Calendar.tsx
Line 19:21: Unnecessary escape character: \. no-useless-escape