Repository navigation
Feature: Add line/links to error messages #7045
Description
Is this a bug report?
No
How it is now
I have an error message like this:
Compiled with warnings.
./src/components/installation/Websocket.tsx
Line 77: Expected to return a value at the end of arrow function array-callback-return
Search for the keywords to learn more about each warning.
To ignore, add // eslint-disable-next-line to the line before.
I can cmd+click on ./src/components/installation/Websocket.tsx and it'll take me to the right file ⭐️
What would be nice is that in the case where there is one error per file, it could instead be ./src/components/installation/Websocket.tsx:77 and clicking on that in the terminal would take directly there. The :77 could be colored using ANSI to be the background color, so it's not visible.
Alternatively, and perhaps a nicer UI, is to use hyperlinks one the Line 77 using something like hyperlinker which would have the href as ./src/components/installation/Websocket.tsx:77. This works in quite a lot of terminals now.
Activity
heyimalex commentedon May 13, 2019
Related to #6980 I think, though the hyperlinker suggestion is new.
justingrant commentedon May 21, 2019
If we do this, please make sure to include the column number too, not just the line number, so that editors can navigate to the exact row/col position. I believe the usual format is something like
/foo/bar.js:287:13which will navigate to line 287, column 13 of bar.js.See https://www.gnu.org/prep/standards/html_node/Errors.html