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

Include location list end column for rust lint #602

Merged
merged 3 commits into from
May 31, 2017
Merged

Include location list end column for rust lint #602

merged 3 commits into from
May 31, 2017

Conversation

jonhoo
Copy link
Contributor

@jonhoo jonhoo commented May 30, 2017

Fixes #599.

I've updated the tests in test/handler/test_rust_handler.vader by extracting the encountered values from the failed tests. Where does src/playpen.rs come from?

@w0rp
Copy link
Member

w0rp commented May 31, 2017

I'll have to look into how to add highlights which span multiple lines to support end_lnum.

@w0rp
Copy link
Member

w0rp commented May 31, 2017

Highlighting now supports problems spanning many lines, so this should all just work now.

@w0rp w0rp merged commit 5eb80f0 into dense-analysis:master May 31, 2017
@w0rp
Copy link
Member

w0rp commented May 31, 2017

Cheers! 🍻

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

Successfully merging this pull request may close these issues.

2 participants