[REF] javascript-lint: Use eslint instead of jshint

[moylop260]

http://eslint.org/ is a tool better than jshint.
Good post about:

• https://www.sitepoint.com/comparison-javascript-linting-tools/

FYI this tool requires a upgrade of nodejs version 6 installing for travis with: nvm install 6
This step spent +2 minutes to current time of the lint build.

• Disable checks for /doc and /templates folder by @yvaucher request: [IMP] javascript-lint: Enable jshint configuration file #49 (comment)

• ToDo: After approval this one we will need add nvm upgrade from MQT

For merger

• Before merge this one is required merge the following PR: [REF] maintainer-quality-tools: Use eslint instead of jshint for javascript lints maintainer-quality-tools#389

Recovered from #87

[moylop260]

ping

[pedrobaeza]

I understand that we need to add a .jslintrc file in MQT, isn't it?

[moylop260]

Yes, that is right
And install nvm too
More info: https://github.com/OCA/maintainer-quality-tools/pull/389/files

[oscarolar]

👍

[yvaucher]

Thanks for taking into account my request.

What is required to ignore the doc folder then ?
Is there a config file to add ?

I'm a bit concerned about the +2 min on each builds, this seems a lot to me.

[moylop260]

@yvaucher
Thanks for your approve although your questions.

What is required to ignore the doc folder then ?

Generally the doc folder has js files based on templates that we don't want fix lints.
But if you have a good case for re-enable we don't have problem. (Sorry I didn't understand the question...)

I'll create the change to ignore them.

Is there a config file to add ?

Yes, .jslintrc

I'm a bit concerned about the +2 min on each builds, this seems a lot to me.

Yes me too, but this is a issue of self-travis that have a too old version nodejs installed by default and eslint requires newest 6.latest version
Maybe we can create a new issue for travis-ci in order to request a update of nodejs for the default image used or we could use a way to cache it but I don't have idea how to do it.
The ideas are welcome.

[moylop260]

@yvaucher
Added doc and template directory names to skipping list.