SonarQube JavaScript
  1. SonarQube JavaScript
  2. SONARJS-77

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: 1.1
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      0

      Issue Links

        Activity

        Hide
        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
        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
        Evgeny Mandrikov added a comment -

        Done.

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

        Manually tested !

        Show
        Freddy Mallet added a comment - Manually tested !

          People

          • Assignee:
            Evgeny Mandrikov
            Reporter:
            Evgeny Mandrikov
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: