search

woldner / VisualLinter

Visual Studio Linter using ESLint.
MIT License
25 stars 3 forks source link

[babel-eslint/v11] Specified argument was out of the range of valid values. Parameter name: lineNumber #53

Closed ivocoumans closed 5 years ago

ivocoumans commented 5 years ago

Installed product versions

Description

When running the extension, the following error is shown in the Output Window: Specified argument was out of the range of valid values. Parameter name: lineNumber

Steps to recreate

  1. Install the extension
  2. Run Visual Studio
  3. Open the Output Window

Current behavior

Instead of running the linter, only the error is shown in the Output Window.

Expected behavior

It should run the linter.

woldner commented 5 years ago

Hi @icoumans could you create a repo for me to test against? The exception you're getting is tightly coupled with the contents of the file being linted. It would help with debugging.

Thank you

ivocoumans commented 5 years ago

Hi @icoumans could you create a repo for me to test against? The exception you're getting is tightly coupled with the contents of the file being linted. It would help with debugging.

Thank you

I really should have thought about my recent changes before posting this issue. I've been upgrading some packages to use the latest experimental proposals. It looks like the 'culprit' is babel-eslint 11.0.0-beta.0. I don't know whether this issue belongs to VisualLinter or babel-eslint.

When I downgrade or disable babel-eslint, the issue no longer appears.

I have attached a simple test project which demonstrates the issue (note that I have not included any babel config - the issue appears regardless).

TestRepo.zip

woldner commented 5 years ago

Thank you for the test repo. I'll have a look tonight.

When I disable the plugin, or downgrade the version, the issue no longer appears.

When you say disable the plugin, or downgrade the version do you mean babel-eslint?

ivocoumans commented 5 years ago

When you say disable the plugin, or downgrade the version do you mean babel-eslint?

Yes, babel-eslint.

woldner commented 5 years ago

When you say disable the plugin, or downgrade the version do you mean babel-eslint?

Yes, babel-eslint.

Ok, It might be that they've changed something (probably index starting values) in v11 of eslint. I'll have to check. 😸

woldner commented 5 years ago

OK, I am able to reproduce this and like you said, its a compatibility problem with babel-eslint@^11. This is something I will have to look further into, but unfortunately don't have the time currently. I will keep this issue open.

As a workaround you can downgrade babel-eslint to version 10.

woldner commented 5 years ago

Sorry everyone; but the development of this extension has been suspended. You can read more about it here: https://github.com/jwldnr/VisualLinter#deprecated