[katello-devel] New javascript lint run added to katello-unit

Lukas Zapletal lzap at redhat.com
Wed Aug 24 08:36:33 UTC 2011


On 08/24/2011 12:03 AM, Mike McCune wrote:
> # Check JavaScript syntax
> jsl --conf scripts/jsl-errors-only.conf
> OUT=$?
> if [ $OUT -ge 3 ];then
>    echo ""
>    echo ""
>    echo ""
>    echo "SYNTAX ERROR detected in JavaScript files in
> src/public/javascripts/.  "
>    echo "Run jsl --conf scripts/jsl-errors-only.conf to reproduce locally."
>    echo ""
>    echo ""
>    echo ""
>    exit 1
> fi

Nice, might be good idea to put this on to the run-codeasys.sh. This one 
is run from the katello-codestyle job:

http://hudson.rhq.lab.eng.bos.redhat.com:8080/hudson/view/katello/job/katello-codestyle/

We already have there:

- ruby syntax check
- haml syntax check
- python checkstyle & lint
- some additional code analytics

Please note SASS precompilation and JS asset packaging are done during 
RPM build.

-- 
Later,

  Lukas Zapletal | E32E400A
  RHN Satellite Engineering
  Red Hat Czech s.r.o. Brno




More information about the katello-devel mailing list