I would like Rubocop to ignore lines with comments (just a comment or some code with an end of line comment) when checking if a line is too long. Is there a way to do this?
5 Answers
There is a way to ignore cops on a per line basis.
There is also a way to do it via configuration file.
Run rubocop --auto-gen-config
and it will generate a file that you can use to disable the offenses.
The command also gives a hint on what to do to load those options.
On a line per line basis, you can enable and disable the cops as well.
# rubocop:disable RuleByName
This is a long line
# rubocop:enable RuleByName
You can also do more than one rule at a time in your code.
# rubocop:disable BlockComments, AsciiComments
By using an inline directive, the directive becomes valid only for that line, and it would look like this:
# Thanks to @jnt30 for the comment!
method(argument) # rubocop:disable SomeRule, SomeOtherRule
You can read a ton more about RuboCop in its official manual.
To find all the rule names its worth looking in the rubocop config files
cyberwiz says - "run rubocop -D
when I need the rule names rather than looking in the documentation." Update: This is now the default behavior without the flag.
The -D
is now default, so we would get that for "free" now.
-
1Well, the comments may explain the deviation from a style that has been accepted by the team, so this is not a bad thing, right? Otherwise you place it in the rubocop.yml file, and then it is not an accepted style exception, and doesn't need a comment. The comment says "I meant to do that!". Not a bad thing at all.– vgoffCommented Oct 17, 2014 at 17:28
-
2comments are not code, so checking them is semantically different and IMO rubocop should treat it that way.– phoetCommented Oct 17, 2014 at 17:52
-
2Comments are a part of code, and when you deal with code in e-mail or on a terminal. I think that it is bad taste to not have your comments adhere to the same line lengths that have been adopted by "the team" as the code. They shouldn't disrupt the flow just because they are comments. I am sure that rubocop doesn't check comments for anything, other than directives, semantically (meaningfully). It does check line length and the style of comments. So no, it isn't looking for meaning, it is only checking style. Don't discount that "comments are not code" doesn't have to be.– vgoffCommented Oct 18, 2014 at 21:15
-
1Another thing to note, you can also use an "inline" disable by terminating your line with the exclusion so you don't have to enabled it again. Doing so will only disable that rule for the given line, not all coming after it.– jnt30Commented Oct 19, 2016 at 21:22
-
1@Twiek is there anything missing from this answer that you are looking for?– vgoffCommented Oct 24, 2016 at 17:05
It's possible to define regex patterns to automatically ignore certain lines in rubocop.yml
, so you could choose to ignore all lines starting with a #
character:
Layout/LineLength:
Max: 80
AllowedPatterns: ['\A#']
Or:
Layout/LineLength:
Max: 80
AllowedPatterns:
- !ruby/regexp /\A#/
This could be improved so that "indented" comment lines (i.e. whitespace followed by a #
character) are also ignored, if that's what you want. e.g.
Layout/LineLength:
Max: 80
AllowedPatterns:
- !ruby/regexp /\A *# /
Note that this doesn't account for lines of code that end with a comment, though:
some_code(that_does_something) # This line would NOT be ignored by Rubocop.
-
14You can expand that regexp by include lines that can have whitespaces:
IgnorePatterns: ['(\A|\s)#']
Commented Nov 10, 2017 at 13:21 -
3Thanks @poustovitss. There's a typo: it should be
IgnoredPatterns
insteadIgnorePatterns
(it's missing the letter 'd').– HoracioCommented Dec 11, 2017 at 22:44 -
3I used:
IgnoredPatterns: ['^ *# ']
This allows full-line comments with any indentation. It does not ignore code with trailing comments, which is my preference. Commented Sep 24, 2020 at 19:50 -
2Warning: obsolete parameter
IgnoredPatterns
(forLayout/LineLength
) found in .rubocop.ymlIgnoredPatterns
has been renamed toAllowedPatterns
. <- for latest versions– dcanguloCommented Apr 30, 2022 at 2:41 -
1
You can use the following comment with rubocop to ignore a specific rule:
# rubocop:disable Layout/LineLength
def this_could_be_a_very_long_line_that_extends_forever_into_infinity
end
# rubocop:enable Layout/LineLength
You can also ignore whole files by adding them to .rubocop.yml
:
AllCops:
Exclude:
- path/to/file.rb
-
2
i think the basic idea here is that you want to enforce line length, no matter what is after n characters. the default to 80 characters is some cargo cult for old terminal windows that could only hold that number of chars. the only option that i saw in the code is an option to allow urls that might exceed the character limit.
you can ignore whole files, i guess that's not what you are looking for.
-
8These days, the idea behind 80 chars isn't so much "cargo cult" for the terminal, there's still a logical reason for it: anyone can split their editor or IDE windows however they want, and as long as they're just wider than 80 characters, they won't need to change the width or experience wrapping. Commented Feb 17, 2016 at 15:38
-
2IMO if you don't have an IDE that supports soft wrapping, your tooling is not up to date.– phoetCommented Feb 20, 2016 at 8:55
-
880 chars is also pretty readable, whereas 40 or 200 is less so, so it's also a usability thing Commented Apr 18, 2016 at 7:43
-
#1 "as long as they're just wider than 80 characters" 800x600 is wider than 80 chars, and you have same problem viewing side-by-side 2 80 chars files. This is nonsense and only applies to 1366 res. With 1920+ taking over, the next argument will be "I can split 3, 4 files" Commented Jul 5, 2017 at 14:12
-
1we had it at 80, then changed it to 120. Looking at PR's while doing code review on Github in a split-screen view on 13" laptops became a problem. we switched back to 80. Commented Oct 7, 2019 at 15:55
The following configuration worked for me:
Layout/LineLength:
AllowedPatterns: ['^(\s*#)']
This regex only works when the entire line is commented out. Code followed by a long comment on the same line will still trigger a Rubocop lint error, which is by design.