#7863: Remove lint from auxiliary JS files
------------------------+---------------------------------------------------
Reporter: mpatel | Owner: was
Type: defect | Status: needs_review
Priority: minor | Milestone: sage-4.3.1
Component: notebook | Keywords:
Work_issues: | Author: Mitesh Patel
Upstream: N/A | Reviewer:
Merged: |
------------------------+---------------------------------------------------
Changes (by mpatel):
* status: new => needs_review
Old description:
> [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/
New description:
[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 disabled [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 present architecture 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#comment:1>
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.
