Sonar Plugins

Do not use jslint4java

Details

  • Type: Task Task
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: JAVASCRIPT-1.0
  • Component/s: JavaScript
  • Labels:
    None
  • Number of attachments :
    3

Description

JSLint should be fully replaced by own rule engine (SONARPLUGINS-1788).
And here is some reasons to not use JSLint:

Issue Links

relates to

New Feature - A new feature of the product, which has yet to be developed. SONARPLUGINS-1788 Provide JavaScript rule engine based on SSLR

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.
supercedes

Bug - A problem which impairs or prevents the functions of the product. SONARPLUGINS-1546 Misspelled property prefix

  • Minor - Minor loss of function, or other problem where easy workaround is present.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

New Feature - A new feature of the product, which has yet to be developed. SONARPLUGINS-1670 Would like to pick either JSLint or JSHint as my code analysis tool

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Improvement - An improvement or enhancement to an existing feature or task. SONARPLUGINS-1045 Update jslint4java to version 2.0.2

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Evgeny Mandrikov made changes -
Field Original Value New Value
Link This issue relates to SONARPLUGINS-1788 [ SONARPLUGINS-1788 ]
Evgeny Mandrikov made changes -
Attachment jslint-stopping-error.png [ 59910 ]
Evgeny Mandrikov made changes -
Attachment jslint-stopping-error.png [ 59910 ]
Evgeny Mandrikov made changes -
Attachment jslint-stopping-error.png [ 59911 ]
Evgeny Mandrikov made changes -
Attachment jslint-line-too-long.png [ 59912 ]
Evgeny Mandrikov made changes -
Attachment jslint-other-rules.png [ 59913 ]
Evgeny Mandrikov made changes -
Description Here is some reasons to not use JSLint:
* Almost always stops execution before reporting all violations - [^jslint-stopping-error.png]. Note that this leads to side effects in project history.
* Quality of rules leaves much to be desired - [^jslint-line-too-long.png].
* No granularity of rules and thus no control via profile - [^jslint-other-rules.png].
Evgeny Mandrikov made changes -
Description Here is some reasons to not use JSLint:
* Almost always stops execution before reporting all violations - [^jslint-stopping-error.png]. Note that this leads to side effects in project history.
* Quality of rules leaves much to be desired - [^jslint-line-too-long.png].
* No granularity of rules and thus no control via profile - [^jslint-other-rules.png].
Here is some reasons to not use JSLint:
* Almost always stops execution before reporting all violations - [^jslint-stopping-error.png]. Note that this leads to side effects in project history.
* Quality of rules leaves much to be desired - [^jslint-line-too-long.png].
* No granularity of rules and thus no control via profile - [^jslint-other-rules.png].
* Size of dependencies - around 1 MB.
Evgeny Mandrikov made changes -
Description Here is some reasons to not use JSLint:
* Almost always stops execution before reporting all violations - [^jslint-stopping-error.png]. Note that this leads to side effects in project history.
* Quality of rules leaves much to be desired - [^jslint-line-too-long.png].
* No granularity of rules and thus no control via profile - [^jslint-other-rules.png].
* Size of dependencies - around 1 MB. 		JSLint should be fully replaced by own rule engine (SONARPLUGINS-1788).
And here is some reasons to not use JSLint:
* Almost always stops execution before reporting all violations - [^jslint-stopping-error.png]. Note that this leads to side effects in project history.
* Quality of rules leaves much to be desired - [^jslint-line-too-long.png].
* No granularity of rules and thus no control via profile - [^jslint-other-rules.png].
* Size of dependencies - around 1 MB.
Evgeny Mandrikov made changes -
Link This issue supercedes SONARPLUGINS-1670 [ SONARPLUGINS-1670 ]
Evgeny Mandrikov made changes -
Fix Version/s JAVASCRIPT-0.5 [ 18248 ]
Evgeny Mandrikov made changes -
Link This issue supercedes SONARPLUGINS-1045 [ SONARPLUGINS-1045 ]
Evgeny Mandrikov made changes -
Link This issue supercedes SONARPLUGINS-1546 [ SONARPLUGINS-1546 ]
Hide
Permalink
Evgeny Mandrikov added a comment -

Done.

Show
Evgeny Mandrikov added a comment - Done.
Evgeny Mandrikov made changes -
Status Open [ 1 ] Resolved [ 5 ]
Assignee Evgeny Mandrikov [ godin ] Freddy Mallet [ freddy.mallet ]
Resolution Fixed [ 1 ]
Hide
Permalink
Freddy Mallet added a comment -

Manually tested !

Show
Freddy Mallet added a comment - Manually tested !
Freddy Mallet made changes -
Assignee Freddy Mallet [ freddy.mallet ] Evgeny Mandrikov [ godin ]
Evgeny Mandrikov made changes -
Status Resolved [ 5 ] Closed [ 6 ]

People

  • Assignee:
    Evgeny Mandrikov
    Reporter:
    Evgeny Mandrikov
Vote (1)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: