Add ignore_comments param to MaxLineLength check

[Nezteb]

Long story short, a lot of projects I work with sometimes have very long comment lines and I want to ignore them in a opt-in fashion for this Credo check. 🙏

[rrrene]

Hi, sorry for the delay.

I will include this in Credo 1.8, just need a weekend or two more to get things into RC territory 👍

On a side note, I want to say that this check is becoming the most configurable (is that a word?) while it should be obsolete when using the formatter, which is kind of ironic 👀

[Nezteb]

I will include this in Credo 1.8, just need a weekend or two more to get things into RC territory 👍

No rush at all! 🙏

I want to say that this check is becoming the most configurable (is that a word?) while it should be obsolete when using the formatter

Yeah some folks I've talked to prefer to disable this check altogether because of how Elixir's formatter treats line length config:

• mix format produces too long lines as specified in config elixir-lang/elixir#7851 (comment)
• Mix format exceed line length at == elixir-lang/elixir#9809 (comment)
• mix format does not respect max line length elixir-lang/elixir#11194 (comment)
• Formatter breaking line_length in guards elixir-lang/elixir#13418 (comment)

Still, I opted for adding a configuration option. 😄