Salsa could show warning messages on require statements if it is not configured with a module system

[egamma]

From @alexandrudima on January 27, 2016 10:36

Testing #2218

I don't have a jsconfig.json

Copied from original issue: microsoft/vscode#2426

[billti]

This is valid CommonJS/Node code and doesn't need to be compiled/transpiled. It seems reasonable this should just work without a config file.

[alexdima]

@billti The reason I filed this is because from the past, I have noticed many users do not even realize they need to create a jsconfig.json file to get good language support. And many folks don't read the documentation so they just jump in without a jsconfig.json file and see that cross-file suggestions don't work. Would be nice if we have a path to guide them towards adding it...

[billti]

We're looking into making the defaults (with or without a config file) work as part of #6690. If we switch to 'commonjs' by default, this issue should evaporate.

EDIT: We just agreed to make CommonJS the default if not specified.