#7863: Remove lint from auxiliary JS files
------------------------+---------------------------------------------------
Reporter: mpatel | Owner: was
Type: defect | Status: new
Priority: minor | Milestone: sage-4.3.1
Component: notebook | Keywords:
Work_issues: | Author: Mitesh Patel
Upstream: N/A | Reviewer:
Merged: |
------------------------+---------------------------------------------------
[http://www.jslint.com/ JSLint] on "The Good Parts" setting, applied to
`sagenb/data/sage/js/*.js` but ''not'' `notebook_lib.js`. The latter will
have its own ticket.
For now, I've have not enabled [http://ejohn.org/blog/ecmascript-5-strict-
mode-json-and-more/ strict mode], since most (all?) current browsers don't
yet have it.
Given the overall current design of the notebook JS library --- use lots
of global variables, etc. --- I haven't implemented ''all'' of JSLint's
suggestions. More generally:
"If you're writing javascript code, [http://www.jslint.com/ JSLint] is
a really fine piece of software, too. You don't have to follow its
recommendations blindly, but understanding what it says about your code
can greatly improve your skills." -- http://jsbeautifier.org/
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7863>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
