572259 - Flag errors in the user's code

Status: RESOLVED INVALID

(Skywriter Graveyard :: Editor, defect)

Description

[Kevin Dangoor]

Provide a mechanism for displaying errors in user entered code. Minimally, this can be something like running JSLint on save. Optimally, it would be more like an IDE that flags errors soon after they're typed.

Comment 1

[Patrick Walton (:pcwalton)]

Part 1: Import JSLint into our tree:
http://hg.mozilla.org/labs/bespinclient/rev/780d540a1ca1

Comment 2

[Patrick Walton (:pcwalton)]

Part 2: Add a "jslint" command to run JSLint:
http://hg.mozilla.org/labs/bespinclient/rev/5871e6a029af

Comment 3

[Patrick Walton (:pcwalton)]

Part 3: Introduce save hooks to allow plugins to perform tasks on save
http://hg.mozilla.org/labs/bespinclient/rev/46b17280cf9e

Comment 4

[Patrick Walton (:pcwalton)]

Part 4: Create a JSLint save hook
http://hg.mozilla.org/labs/bespinclient/rev/19b3a4bad823

Comment 5

[Kevin Dangoor]

Looking good so far. A couple of comments:

1. ultimately, jslint_commands shouldn't depend on file_commands. I do seem to recall that we have to do that presently, though, to ensure that the savehook extension point exists.

2. it would be good to use the notifier if there are any errors (so that the user gets a popup notification that something is amiss). That should be for syntax error level errors only.

Comment 6

[Kevin Dangoor]

ACETRANSITION

The Skywriter project has merged with Ajax.org's Ace project (the full server part of which is their Cloud9 IDE project). Background on the change is here:

http://mozillalabs.com/skywriter/2011/01/18/mozilla-skywriter-has-been-merged-into-ace/

The bugs in the Skywriter product are not necessarily relevant for Ace and quite a bit of code has changed. For that reason, I'm closing all of these bugs. Problems that you have with Ace should be filed in the Ace issue tracker at GitHub:

https://github.com/ajaxorg/ace/issues