Dash causes linter to not underline whole error

[OmgImAlexis]

If there's an issue with importing a js package only the first half of the package name is highlighted when there's a dash in it.

Ref: steelbrain/linter#1440

---

IssueHunt Summary

Sponsors (Total: $40.00)

• issuehunt ($40.00)

Become a sponsor now!

Or submit a pull request to get the deposits!

Tips

• Checkout the Issuehunt explorer to discover more funded issues.
• Need some help from other developers? Add your repositories on IssueHunt to raise funds.

[Arcanemagus]

At it's absolute root, this is a combination ESLint and Atom issue.

Some ESLint rules give a full start line:col and end line:col, allowing the range highlighted to come straight from ESLint. The problem is that most ESLint rules only return a single line:column as the location for the rule, which means some guessing as to the end of the range has to be made. When this is all the information that is available the fallback to to use the regex from Atom's language-javascript that defines a "word" to determine how long to make the range. In this case it seems the definition of "word" stops on a - character.

I know there is some effort to majorly expand the number of rules that ESLint provides a range result for, I'm not sure how outdated xo is on the version of ESLint it bundles so it's possible an update of that will fix some cases. The "word" definition from Atom unfortunately most likely can't be changed since it must be very generic (It's not just words in strings).

tl;dr: The best that can be done about this is already being done, and time will fix this in more and more cases.

[Arcanemagus]

Ah, looks like the "major expansion of range results" will be in ESLint v4: eslint/eslint#3307 (comment).

[IssueHuntBot]

@issuehunt has funded $40.00 to this issue.

---

• Submit pull request via IssueHunt to receive this reward.
• Want to contribute? Chip in to this issue via IssueHunt.
• Checkout the IssueHunt Issue Explorer to see more funded issues.
• Need help from developers? Add your repository on IssueHunt to raise funds.

[sindresorhus]

Closing as Atom is abandoned.