You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Lines where obeying the column limit is not possible (for example, a long URL in Javadoc, or a long JSNI method reference).
But as you can see google style guide says we only need to ignore the long JSNI method references. One of the maintainer suggested that the JSNI method reference is wrap-able so why there is a need to ignore JSNI reference for this rule when we can wrap it to make it under the column limit?
Here is the conversation link for your reference. Can anyone explain us how the Column Limit rule should work for JSNI Methods? Thank you.
The text was updated successfully, but these errors were encountered:
Hey, I'm a contributor at Checkstyle. I was working on this issue to add support for detecting JSNI method for rule 4.4 Column Limit: 100. I made a PR regarding this issue, but there I got stuck on how we should approach this issue. Initially I discussed with the maintainers and we came up with the idea of ignoring the column limit for whole JSNI method declaration for this rule. You can take a look at this file for better understanding how we are ignoring JSNI methods for this rule: https://github.com/checkstyle/checkstyle/blob/40934dddede18b5f004f678f6704a84e32847cb2/src/it/resources/com/google/checkstyle/test/chapter4formatting/rule44columnlimit/InputLineLengthJsniMethods.java
But as you can see google style guide says we only need to ignore the long JSNI method references. One of the maintainer suggested that the JSNI method reference is wrap-able so why there is a need to ignore JSNI reference for this rule when we can wrap it to make it under the column limit?
Here is the conversation link for your reference. Can anyone explain us how the Column Limit rule should work for JSNI Methods? Thank you.
The text was updated successfully, but these errors were encountered: