search

brackets-archive / bracketsIssues

Archive of issues in brackets.
0 stars 0 forks source link

[CLOSED] Add jslint #170

Open core-ai-bot opened 3 years ago

core-ai-bot commented 3 years ago

Issue by gruehle Monday Jan 30, 2012 at 17:44 GMT Originally opened as https://github.com/adobe/brackets/pull/166

@njx

This pull request adds a new "Enable JSLint" item in the Debug menu, which is checked by default. When checked, any time a JS or HTML file is activated or saved, the contents are run through JSLint. Errors (if any) are displayed at the bottom of the code window. Clicking an error moves the cursor to the error location. If there are no JSLint errors, a gold star appears in the upper-right corner.

By default, no options are passed to JSLint. You must specify any options in a jslint comment in your source file.

There is one pre-processing step that is done before JSLint is run - any lines that are pure whitespace are replaced with an empty (no whitespace) line. This is because JSLint returns an error if there is any additional whitespace at the end of the line, even if the line is pure whitespace.

This pull request makes brackets.js and strings.js pass JSLint. The other files will be fixed in future pull requests.

gruehle included the following code: https://github.com/adobe/brackets/pull/166/commits

core-ai-bot commented 3 years ago

Comment by njx Monday Jan 30, 2012 at 19:25 GMT

Done with review.

core-ai-bot commented 3 years ago

Comment by gruehle Monday Jan 30, 2012 at 20:44 GMT

All comments have been resolved.