SublimeLinter / SublimeLinter-luacheck

SublimeLinter 3 plugin for lua, using luacheck
MIT License
28 stars 9 forks source link

Error code fixes #17

Closed Resike closed 6 years ago

kaste commented 6 years ago

Can you post some typical output of lua check, please?

Resike commented 6 years ago

SublimeLinter 4 has an function to show error/warning codes, without this fix the error/warning output ode is only W: image

kaste commented 6 years ago

Ah that's cool, does it work for errors as well.

Can you still post the raw output from the linter.

Resike commented 6 years ago

Raw print from python, also put some errors to it: sublime_text_2018-03-19_23-10-52

kaste commented 6 years ago

Sorry I don't get this regex: Shouldn't it be

^.+:(?P<line>\d+):(?P<col>\d+)\-(?P<col_end>\d+): \((?:(?P<error>E\d+)|(?P<warning>W\d+))\) (?P<message>.+)

That is, both error and warning (named) group match \d+

https://regex101.com/r/JFyiaZ/1

Resike commented 6 years ago

Yeah sorry I left the warning one out. And for whatever reason it was working properly like that too.

kaste commented 6 years ago

Okay, let's go.

kaste commented 6 years ago

🎉

Released as https://github.com/SublimeLinter/SublimeLinter-luacheck/releases/tag/2.1.1