SublimeLinter / SublimeLinter-luacheck

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

Fix passing a globals list to luacheck #1

Closed henriquegemignani closed 10 years ago

henriquegemignani commented 10 years ago

luacheck receives a comma-delimited list of globals

aparajita commented 10 years ago

Are you absolutely sure about this? The docs clearly show space-delimited args. But I haven't actually tested on the command line. Can you show the command line output for both?

henriquegemignani commented 10 years ago

Checking things, what the patch does it not actually create a comma-delimited list of globals, but instead tell SublimeLinters to not merge the list into a single argument and instead pass each item as a different positional argument.

Also, the --globals argument is similar to the --ignore argument, so both should be treated the same way.

aparajita commented 10 years ago

Okay.