Details

    • Type: New Feature New Feature
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.4
    • Component/s: Rules
    • Labels:
      None
    • Number of attachments :
      0

      Issue Links

        Activity

        Hide
        Dinesh Bolkensteyn added a comment -

        Done

        Show
        Dinesh Bolkensteyn added a comment - Done
        Hide
        Fabrice Bellingard added a comment -

        Perfect Dinesh!

        I just reopen the ticket because there was a typo in the existing description: "if" should be replaced by "is" in "// The following line if compliant with the default configuration of the rule" (I updated RSPEC).

        Please close directly the ticket once this is done.

        Show
        Fabrice Bellingard added a comment - Perfect Dinesh! I just reopen the ticket because there was a typo in the existing description: "if" should be replaced by "is" in "// The following line if compliant with the default configuration of the rule" (I updated RSPEC). Please close directly the ticket once this is done.
        Hide
        Dinesh Bolkensteyn added a comment -

        Fixed

        Show
        Dinesh Bolkensteyn added a comment - Fixed
        Hide
        Dinesh Bolkensteyn added a comment -

        After the deployment on Dory, I notice several issues on Javascript such as:

        b.sequence(/* no line terminator here */SPACING_NO_LB, NEXT_NOT_LB, EXPRESSION, EOS),
        

        This seems to be a false positive

        Show
        Dinesh Bolkensteyn added a comment - After the deployment on Dory, I notice several issues on Javascript such as: b.sequence(/* no line terminator here */SPACING_NO_LB, NEXT_NOT_LB, EXPRESSION, EOS), This seems to be a false positive
        Hide
        Dinesh Bolkensteyn added a comment -

        Fixed

        Show
        Dinesh Bolkensteyn added a comment - Fixed
        Hide
        Fabrice Bellingard added a comment -

        Indeed, well spotted. Tested!

        Show
        Fabrice Bellingard added a comment - Indeed, well spotted. Tested!

          People

          • Assignee:
            Dinesh Bolkensteyn
            Reporter:
            Dinesh Bolkensteyn
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: