168

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 5

287

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.

9
  • 1
    Well, 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.
    – vgoff
    Commented Oct 17, 2014 at 17:28
  • 2
    comments are not code, so checking them is semantically different and IMO rubocop should treat it that way.
    – phoet
    Commented Oct 17, 2014 at 17:52
  • 2
    Comments 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.
    – vgoff
    Commented Oct 18, 2014 at 21:15
  • 1
    Another 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.
    – jnt30
    Commented Oct 19, 2016 at 21:22
  • 1
    @Twiek is there anything missing from this answer that you are looking for?
    – vgoff
    Commented Oct 24, 2016 at 17:05
52

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.
7
  • 14
    You can expand that regexp by include lines that can have whitespaces: IgnorePatterns: ['(\A|\s)#'] Commented Nov 10, 2017 at 13:21
  • 3
    Thanks @poustovitss. There's a typo: it should be IgnoredPatterns instead IgnorePatterns (it's missing the letter 'd').
    – Horacio
    Commented Dec 11, 2017 at 22:44
  • 3
    I 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
  • 2
    Warning: obsolete parameter IgnoredPatterns (for Layout/LineLength) found in .rubocop.yml IgnoredPatterns has been renamed to AllowedPatterns. <- for latest versions
    – dcangulo
    Commented Apr 30, 2022 at 2:41
  • 1
    @DavidCruwys same thought! super confusing.
    – courtsimas
    Commented Aug 11, 2022 at 2:49
24

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
1
  • 2
    It is # rubocop:disable Layout/LineLength now
    – Yathi
    Commented Oct 9, 2020 at 15:25
5

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.

6
  • 8
    These 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
  • 2
    IMO if you don't have an IDE that supports soft wrapping, your tooling is not up to date.
    – phoet
    Commented Feb 20, 2016 at 8:55
  • 8
    80 chars is also pretty readable, whereas 40 or 200 is less so, so it's also a usability thing
    – Toni Leigh
    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
  • 1
    we 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
0

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.

Not the answer you're looking for? Browse other questions tagged or ask your own question.