Uploaded image for project: 'translate5'
  1. translate5
  2. TRANSLATE-1244

Include jslint into build process

XMLWordPrintable

    • Icon: Task Task
    • Resolution: Fixed
    • None
    • None
    • None

      problem

      It happens sometimes that JS code is published which is either invalid or hard to understand or does have other problems. Mainly ES6 shorthand method definition which is not allowed in IE11.

      solution

      To prevent this jslint could be used to check the JS code before release (or even before commit on each machine, or nightly on master.translate5.net).

      Installation of jslint is easy, just install nodejs, then do:
      sudo npm install -g jshint

      After that the projectfiles can be checked with:

      jshint public/modules/ application/modules/

      Which currently gives 251 errors in Plugin and core JS code.

      Most of the stuff can be fixed easily, also is jslint configurable, so we can configure what should be threatened as error and what not.

            tlauria Thomas Lauria
            tlauria Thomas Lauria
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: