Jackmcbarn wrote:
There are no false positives at all, since it tests it by actually loading it into Lua.
Does the checkbox currently only allow overriding the type of error that would completely prevent execution of a module? I encounter a lot of code linters and syntax highlighters that mess up or get confused. It's difficult for me to understand what exactly "with errors" means here.
I don't know Lua well enough to use it as an example, but as James mentioned CSS, using the example of ".foo { -moz-colour: red;", to me there's probably a distinction to be made between the invalid "-moz-colour" property and the lack of a "}". Though perhaps certain clients wouldn't mind either, which brings us back to the meaning of error.
I think a total disallow is warranted because letting the page be saved with such an error means that the entire module is 100% useless (and will break every page that uses it) until someone fixes it.
This kind of assumes a bad scenario (breaking pages), which doesn't really apply to making a new module or improving an existing module in a sandbox page. For me, it doesn't seem very difficult to envision a scenario in which not being able to save currently broken code would be annoying, though it's quite possible my concern about this is simply overblown.
MZMcBride