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

Detect when a linter is not installed #352

Open
xelaadryth opened this issue Sep 13, 2018 · 0 comments
Open

Detect when a linter is not installed #352

xelaadryth opened this issue Sep 13, 2018 · 0 comments

Comments

@xelaadryth
Copy link
Collaborator

If a linter is not installed and you don't run with --install or --autorun, then we don't install the linter and the run will fail with an unhelpful error. For instance, with just jsonlint enabled but not installed:

> inline-plz --dryrun
Invalid message: The system cannot find the path specified.

If we add a "detect installation location" field to each linter to see if they're installed, we can check for file or directory presence and throw an exception instead of attempting to subprocess run a non-existent executable that gives better clarity on why it's failing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant