Site.js default layout script's code is not compliant with eslint

[yamgent]

Tell us about your environment

• MarkBind Version: 2.2.0

What did you do? Please include the actual source code causing the issue.

markbind/src/Site.js

Lines 170 to 172 in cf57489

	const LAYOUT_SCRIPTS_DEFAULT = 'MarkBind.afterSetup(() => {\n'
	+ ' // Include code to be called after MarkBind setup here.\n'
	+ '});\n';

The default content needs a // eslint-disable-next-line no-undef. In other words, it should be like this:

 const LAYOUT_SCRIPTS_DEFAULT = '// eslint-disable-next-line no-undef\n'
   + 'MarkBind.afterSetup(() => {\n' 
   + '  // Include code to be called after MarkBind setup here.\n' 
   + '});\n'; 

Otherwise, projects that uses eslint (such as MarkBind) will complain about a no-undef error.

[Xenonym]

Is this necessary? It seems strange to assume that sites built with MarkBind would definitely use ESLint.

[marvinchin]

I think it makes sense for the files generated by MarkBind to pass our own ESLint rules.

[yamgent]

Is this necessary? It seems strange to assume that sites built with MarkBind would definitely use ESLint.

We can alternatively update our eslint configuration to disable the check for expected folders if that fits the assumption.