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

Improve error handling for misconfigured instances #835

Open
2 of 4 tasks
byCedric opened this issue Oct 20, 2019 · 1 comment
Open
2 of 4 tasks

Improve error handling for misconfigured instances #835

byCedric opened this issue Oct 20, 2019 · 1 comment

Comments

@byCedric
Copy link
Member

Expected Behavior

While developing new rules, as plugin for example, it might be wise to have some improved error handling. As you can see here, the current error is a syntax error without additional info on what or where it went wrong.

I think this could also benefit other use cases like misconfigured rule settings or config in general. ESLint has some good examples for us to look at, like this one for example.

.eslintrc.js:
        Configuration for rule "semi" is invalid:
        Severity should be one of the following: 0 = off, 1 = warn, 2 = error (you passed '"wrong level"').

It's a rule with the wrong severity level, clearly indicating what is wrong and how to fix it.

Current Behavior

Affected packages

  • cli
  • core
  • prompt
  • config-angular

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

@byCedric
Copy link
Member Author

Closely related to #263

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

No branches or pull requests

1 participant