feature-sliced / steiger

Universal file structure and project architecture linter
https://www.npmjs.com/package/steiger
MIT License
65 stars 4 forks source link

Feature request: Add links to files in console #39

Closed EliseyMartynov closed 2 months ago

EliseyMartynov commented 3 months ago

Привет! В eslint инструментах есть удобная фича перехода по ссылке на файл по подсвеченной проблеме. Очень бы хотелось видеть это в steiger, т.к. это значительно повышает DX. Сейчас путь до файла в консоли есть, но активной ссылки на этот текст нет. Это вносит определенные неудобства.

illright commented 3 months ago

Да, это проблема. Я не знаю пока, как лучше сделать это, потому что некоторые правила ругаются не на один файл, а на несколько, и непонятно, куда линкать. Но ещё не думал об этом, если честно

EliseyMartynov commented 3 months ago

Если идти 1 к 1 (один файл одна ошибка), то если правило ругается на несколько файлов - можно разделить это на три одинаковые ошибки с ссылками на разные файлы

illright commented 3 months ago

Here's a rundown of existing rules and what object is the closest point to the error:

So really, only three rules here can be linked to files, most others can only be linked to a folder. That's not a big deal, we can link to folders and Cmd+Click would open them in the file browser sidebar, but it's not ideal

illright commented 2 months ago

Released Steiger v0.3.0 with this feature included :)

image