Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature request: Add links to files in console #39

Open
EliseyMartynov opened this issue Jun 24, 2024 · 3 comments
Open

Feature request: Add links to files in console #39

EliseyMartynov opened this issue Jun 24, 2024 · 3 comments
Assignees
Labels
PRIORITY | HIGH Important and significant TYPE | enhancement New feature or request

Comments

@EliseyMartynov
Copy link
Member

EliseyMartynov commented Jun 24, 2024

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

@illright
Copy link
Contributor

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

@illright illright added PRIORITY | HIGH Important and significant TYPE | enhancement New feature or request labels Jun 24, 2024
@EliseyMartynov
Copy link
Member Author

EliseyMartynov commented Jun 25, 2024

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

@illright
Copy link
Contributor

illright commented Jun 30, 2024

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

  • ambiguous-slice-names — the slice folder
  • excessive-slicing — layer or slice group
  • forbidden-imports — the file that contains the wrong import
  • import-locality — the file that contains the wrong import
  • inconsistent-naming — layer or slice group
  • insignificant-slice — slice
  • no-file-segments — file segment
  • no-layer-public-api — the index on the layer folder
  • no-processes — layer folder
  • no-public-api-sidestep — the file that contains the wrong import
  • no-reserved-folder-names — reserved folder
  • no-segmentless-slices — slice folder
  • public-api — slice folder
  • repetitive-naming — layer folder
  • segments-by-purpose — segment folder
  • shared-lib-grouping — shared/lib

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 illright self-assigned this Jul 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PRIORITY | HIGH Important and significant TYPE | enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants