Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #4366 (closed)

Opened 13 years ago

Closed 12 years ago

Last modified 12 years ago

Javascript code checker (jslint)

Reported by: jburel Owned by: wmoore
Priority: major Milestone: OMERO-4.4.7
Component: Deployment Version: n.a.
Keywords: n.a. Cc: python-team@…
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: 2012-10-23 (1)

Description (last modified by jmoore)

Qaulity control for JavaScript? code
e.g. JSLint

#9697

I've started using jslint in textmate: http://rondevera.github.com/jslintmate/

Does js checking on every save. Seems like a nice way of encouraging some consistency etc across our js files.

I've configured an options file to my own preferences, but we should probably agree on a project-wide config and commit it somewhere.

I'll put it in a sensible location E.g. webgateway/static/webgateway/js/ and we can discuss.

Change History (8)

comment:1 Changed 12 years ago by jmoore

  • Cc python-team@… added; jmoore cxallan removed
  • Component changed from General to Deployment
  • Milestone changed from Unscheduled to OMERO-Beta4.4
  • Priority changed from minor to major
  • Summary changed from Bug checker for Javascript code to Javascript code checker

comment:2 Changed 12 years ago by jmoore

  • Milestone changed from OMERO-Beta4.4 to OMERO-Beta4.4.1

Something we could add in in the 4.4 series.

comment:3 Changed 12 years ago by jmoore

Referencing ticket #4347 has changed sprint.

comment:4 Changed 12 years ago by jmoore

  • Description modified (diff)
  • Owner set to wmoore
  • Sprint set to 2012-10-23 (1)
  • Summary changed from Javascript code checker to Javascript code checker (jslint)

comment:5 Changed 12 years ago by wmoore

  • Status changed from new to accepted

comment:6 Changed 12 years ago by wmoore

  • Resolution set to fixed
  • Status changed from accepted to closed

comment:7 Changed 12 years ago by Will Moore <will@…>

(In [1813f7732e7c4b1d846fa628984e8dcedc68a249/ome.git] on branch develop) Removing non generic globals from jshint conf and adding them as source inline options (see #4366)

comment:8 Changed 12 years ago by Will Moore <will@…>

(In [f6db017825c2e08796b1f40f55db6a906d4dbec2/ome.git] on branch develop) making webgateway's ome.viewport.js lint free as per jshint (see #4366)

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.70003 sec.)

We're Hiring!