Sonar Plugins

Rule: Do not use HTML comments, because they are not part of EcmaScript specification

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: JAVASCRIPT-1.1
  • Component/s: JavaScript
  • Labels:
    None
  • Number of attachments :
    0

Issue Links

relates to

Improvement - An improvement or enhancement to an existing feature or task. SONARPLUGINS-2043 The Javascript analyser doesn't support the HTML comment embedded into the javascript source code

  • 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
Hide
Permalink
Freddy Mallet added a comment - - edited

Works well Evgeny, could you just replace

  • "Do not use HTML-style comments - they are not part of EcmaScript specification." by "Do not use HTML-style comments as they are not part of EcmaScript specification."
  • and "<!-- Non-Compliant" by "<!-- Non-Compliant -->"
Show
Freddy Mallet added a comment - - edited Works well Evgeny, could you just replace "Do not use HTML-style comments - they are not part of EcmaScript specification." by "Do not use HTML-style comments as they are not part of EcmaScript specification." and "<!-- Non-Compliant" by "<!-- Non-Compliant -->"
Hide
Permalink
Evgeny Mandrikov added a comment - - edited

In fact both forms are Non-Compliant, because they can't span multiple lines and treated just like "//". So I'll simply add second one.

Show
Evgeny Mandrikov added a comment - - edited In fact both forms are Non-Compliant, because they can't span multiple lines and treated just like "//". So I'll simply add second one.
Hide
Permalink
Evgeny Mandrikov added a comment -

Done.

Show
Evgeny Mandrikov added a comment - Done.
Hide
Permalink
Freddy Mallet added a comment -

Manually tested !

Show
Freddy Mallet added a comment - Manually tested !

People

  • Assignee:
    Evgeny Mandrikov
    Reporter:
    Evgeny Mandrikov
Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: