Tslint: max-line-length with template literals

@ashwinr @ajafff @adidahiya @JKillian How would you design the options schema for this while remaining backward compatible?

@ashwinr @ajafff @adidahiya @JKillian Sorry to bump this again, but I'd like to get a PR in. Unless I'm missing something, I've been unable to find a case where the options can be either a number or an object.

@joelday The max-line-length rule already accepts either a number of an object. Take a look at the _Config examples_ and _Schema_ section of the rule's documentation:

"max-line-length": [true, 120]
"max-line-length": [true, {"limit": 120, "ignore-pattern": "^import |^export {(.*?)}"}]

So you should be able to add a new property to the object and it will remain backward compatible. Something like this:

{
  "type": "array",
  "items": {
    "oneOf": [
      {
        "type": "number"
      },
      {
        "type": "object",
        "properties": {
          "limit": {
            "type": "number"
          },
          "ignore-pattern": {
            "type": "string"
          },
          "ignore-multiline-template-literals": {
            "type": "boolean"
          }
        },
        "additionalProperties": false
      }
    ]
  },
  "minLength": 1,
  "maxLength": 2
}

@joelday @reduckted This rule has some problems, I found mainly 2: template literals and Regex. In both cases it is hard or not recommendable to cut these long strings.
It is not about ignore-multiline-template-literals ... even single line template literals are a problem as in the example above.

Then a property like ignore-template-literals would be great.

Could also be possible add support for regex?

even single line template literals are a problem as in the example above.

The option should probably ignore all strings, whether they're single-line, multi-line or template strings.

For now, for a single line template literals, it is possible to use ignore-pattern:
if only space indention are used on the propject:
"max-line-length": [true, {"limit": 120, "ignore-pattern": "^\\s*(.+)$"}]
for both tabs ans spaces:
"max-line-length": [true, {"limit": 120, "ignore-pattern": "^[\t\\s]*(.+)$"}]
to alow single line template literal asignment:
"max-line-length": [true, {"limit": 120, "ignore-pattern": "(.+)"}]

If you intend to implement https://github.com/palantir/tslint/issues/3585, I would strongly recommend the sage advice of @reduckted is heeded, and that such an option should be enabled by default for all strings.

Strings could contain absolutely anything. A linter has no business expressing opinions on them, and a fixer certainly hasn't got any business whatsoever touching them because it can't possibly understand the rules that govern "the set of all possible strings".

Hi, I had the same issue in some projects I was participating in and here is the solution I came up with.
It only ignores strings and template strings but we could figure out what kind of options we would like to have, maybe even regex as @artola mentioned.