Details

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

      Issue Links

        Activity

        Hide
        Freddy Mallet added a comment -

        Works well Evgeny, could you just change the violation message as follows :

        "The line contains x characters which is greater that y authorized"

        Show
        Freddy Mallet added a comment - Works well Evgeny, could you just change the violation message as follows : "The line contains x characters which is greater that y authorized"
        Hide
        Evgeny Mandrikov added a comment -

        Done.

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

        In fact Evgeny, looks like we fire the violation as soon as we encounter the first token exceeding the threshold.

        So with a line whose last character is at column 132 we can have the following violation message :

        "The line contains 83 characters which is greater than 80 authorized."

        Show
        Freddy Mallet added a comment - In fact Evgeny, looks like we fire the violation as soon as we encounter the first token exceeding the threshold. So with a line whose last character is at column 132 we can have the following violation message : "The line contains 83 characters which is greater than 80 authorized."
        Hide
        Evgeny Mandrikov added a comment -

        You're right and I guess that this was a reason of a message, which you asked to change I'll take a look on that.

        Show
        Evgeny Mandrikov added a comment - You're right and I guess that this was a reason of a message, which you asked to change I'll take a look on that.
        Hide
        Evgeny Mandrikov added a comment -

        Should be fixed now.

        Show
        Evgeny Mandrikov added a comment - Should be fixed now.
        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: