Currently, the jspwiki maven build is running JSLINT by default as part of
the mvn clean install.
This generates lots of errors and warnings, on the main jspwiki javascripts
files as well as the fck javascripts.
All these errors are not necessarily indicating *real* js errors. It is
known that jslint is using a very strict set of validation rules. These
generally help to make the js more safe, but are not always indicators of
erroneous code.
For example: JSLint is throwing WARNINGs when you using global variables
(eg BROWSER global vars) defined elsewhere.
EG adding following line to FCKConfig.js, will already kill most of the
JSLINT warnings.
/*global FCKConfig, CTRL, SHIFT, ALT */
I recommend to only run JSLINT on the javascript written dedicated for
JSPWiki. JSLINT should not be run on the external libraries ( such as
mootools.js, prettify.js ) on which we are not having any impact anyway.
On the long run, I propose that we would use JSHINT rather then JSLINT.
JSHINT is more configurable then JSLINT.
(btw -- jshint can also be run via the wro4j maven plugin)
dirk
On Wed, Aug 7, 2013 at 11:31 AM, Harry Metske <harry.met...@gmail.com>wrote:
> I personally never used it, so I won't object removing it ....
>
>
> On 7 August 2013 04:11, Glen Mazza <glen.ma...@gmail.com> wrote:
>
> > Returning to this issue, every time we run mvn clean install, JSPWiki
> > looks bad because of all the fckconfig.js errors at the top[1]. Might it
> be
> > better for us to just delete this file from the project? Because
> > fckconfig.js doesn't work anyway[2], it's from 2008 and needs to be
> > replaced with ckconfig.js (which hopefully does a better job with
> jslint).
> > Also, its name--"fck"config is ugly, as it's too close to an English
> > obscenity.
> >
> > Regards,
> > Glen
> >
> > [1] https://builds.apache.org/job/**JSPWiki/200/console<
> https://builds.apache.org/job/JSPWiki/200/console>
> > [2] http://markmail.org/message/**sb7sockky7o2rzdb<
> http://markmail.org/message/sb7sockky7o2rzdb>
> >
> > On 07/25/2013 11:35 AM, Dirk Frederickx wrote:
> >
> >> Gent's,
> >>
> >> Is there an easy way to perform a build but skip the js & css
> compression
> >> step ?
> >> Would be useful during debugging.
> >>
> >>
> >> Currently, I cant run a fresh build because the yuicompressor maven
> plugin
> >> expects a maven version of 3.0.4 or higher, which I'm not yet running.
> >>
> >>
> >>
> >> dirk
> >>
> >>
> >
>
