Skip to content
This repository has been archived by the owner on Sep 3, 2021. It is now read-only.

Error 'squiggles' don't clear in JSON files. #19

Open
pedro-w opened this issue Sep 18, 2018 · 4 comments
Open

Error 'squiggles' don't clear in JSON files. #19

pedro-w opened this issue Sep 18, 2018 · 4 comments

Comments

@pedro-w
Copy link

pedro-w commented Sep 18, 2018

If I edit a non-tmLanguage JSON file when this extension is enabled, it throws up a lot of problem reports which don't get cleared when I continue typing. For example if I am trying to type { "size" : 0 } I see a series of problems for each of the intermediate steps which don't cleared. Without this extension active they do get cleared when I press the 0 (i.e. when it becomes valid JSON). See attached picture.

Version: 1.27.2 (user setup)
Commit: f46c4c469d6e6d8c46f268d1553c5dc4b475840f
Date: 2018-09-12T16:17:45.060Z
Electron: 2.0.7
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
Architecture: x64
Textmate Languages: 0.9.0

Note this is not a textmate file, so this extension probably should not have been activated (as reported in #17 )
not-clearing-errors

@IronGeek
Copy link

I've reported the problem with JSON errors not being cleared too, here: #14...

@pedro-w
Copy link
Author

pedro-w commented Sep 24, 2018

@IronGeek you're right, sorry I missed that. Should I close this one as a duplicate?

@IronGeek
Copy link

@pedro-w I think you should keep this one open for visibility. The other issue (#14) doesn't specifically mention anything about JSON in the title, so it's pretty easy to miss over this one...

@pedro-w
Copy link
Author

pedro-w commented Sep 24, 2018

OK, I think I've got a fix for this. I will put in a Pull Request hopefully later today.

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

Successfully merging a pull request may close this issue.

2 participants