{"action":"created","issue":{"url":"https://api.github.com/repos/eslint/eslint/issues/9059","repository_url":"https://api.github.com/repos/eslint/eslint","labels_url":"https://api.github.com/repos/eslint/eslint/issues/9059/labels{/name}","comments_url":"https://api.github.com/repos/eslint/eslint/issues/9059/comments","events_url":"https://api.github.com/repos/eslint/eslint/issues/9059/events","html_url":"https://github.com/eslint/eslint/issues/9059","id":247230475,"number":9059,"title":"CR - max-len: {ignoreRegExpLiterals: true} is too permissive","user":{"login":"refack","id":96947,"avatar_url":"https://avatars3.githubusercontent.com/u/96947?v=4","gravatar_id":"","url":"https://api.github.com/users/refack","html_url":"https://github.com/refack","followers_url":"https://api.github.com/users/refack/followers","following_url":"https://api.github.com/users/refack/following{/other_user}","gists_url":"https://api.github.com/users/refack/gists{/gist_id}","starred_url":"https://api.github.com/users/refack/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/refack/subscriptions","organizations_url":"https://api.github.com/users/refack/orgs","repos_url":"https://api.github.com/users/refack/repos","events_url":"https://api.github.com/users/refack/events{/privacy}","received_events_url":"https://api.github.com/users/refack/received_events","type":"User","site_admin":false},"labels":[{"id":185518888,"url":"https://api.github.com/repos/eslint/eslint/labels/breaking","name":"breaking","color":"e11d21","default":false},{"id":45711352,"url":"https://api.github.com/repos/eslint/eslint/labels/enhancement","name":"enhancement","color":"84b6eb","default":true},{"id":255319420,"url":"https://api.github.com/repos/eslint/eslint/labels/evaluating","name":"evaluating","color":"fef2c0","default":false},{"id":45798639,"url":"https://api.github.com/repos/eslint/eslint/labels/rule","name":"rule","color":"fbca04","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2017-08-01T23:22:33Z","updated_at":"2017-08-14T19:36:31Z","closed_at":null,"body":"<!--\r\n    ESLint adheres to the [JS Foundation Code of Conduct](https://js.foundation/community/code-of-conduct).\r\n\r\n    This template is for bug reports. If you are reporting a bug, please continue on. If you are here for another reason, please see below:\r\n\r\n    1. To propose a new rule: http://eslint.org/docs/developer-guide/contributing/new-rules\r\n    2. To request a change: http://eslint.org/docs/developer-guide/contributing/changes\r\n    3. If you have any questions, please stop by our chatroom: https://gitter.im/eslint/eslint\r\n\r\n    Note that leaving sections blank will make it difficult for us to troubleshoot and we may have to close the issue.\r\n-->\r\n\r\n**Tell us about your environment**\r\n\r\n* **ESLint Version:** `4.3.0`\r\n* **Node Version:** `master`\r\n* **npm Version:** `5.3.0`\r\n\r\n**What parser (default, Babel-ESLint, etc.) are you using?**\r\nN/A\r\n\r\n**Please show your full configuration:**\r\nhttps://github.com/nodejs/node/blob/master/.eslintrc.yaml\r\nhttps://github.com/nodejs/node/blob/master/lib/.eslintrc.yaml\r\n\r\n```yaml\r\nmax-len: [error, {code: 80,\r\n                  ignoreRegExpLiterals: true,\r\n                  ignoreUrls: true,\r\n                  tabWidth: 2}]\r\n```\r\n**What did you do? Please include the actual source code causing the issue.**\r\n\r\n<!-- Paste the source code below: -->\r\n```js\r\n01: function fatalError(e) {\r\n02:   if (typeof e.stack === 'string') {\r\n03:     process._rawDebug(e.stack);\r\n04:   } else {\r\n05:     const o = { message: e };\r\n06:     Error.captureStackTrace(o, fatalError);\r\n07:     process._rawDebug(o.stack);\r\n08:   }\r\n09:  if (process.execArgv.some((e) => /^--abort[_-]on[_-]uncaught[_-]exception$/.test(e))) {\r\n10:    process.abort();\r\n11:   }\r\n12:   process.exit(1);\r\n13: }\r\n```\r\n\r\n**What did you expect to happen?**\r\nL09 should error.\r\nEven though we have `ignoreRegExpLiterals: true` such cases should not be allowed. It's not the RegExp that is super long, it's the multiple expressions in that single line that caused it to be too long.\r\nI understand it's not a straightforward problem to solve, but IMHO it could be minimized with some heuristics.\r\n\r\n**What actually happened? Please include the actual, raw output from ESLint.**\r\nNothing (i.e. lint passed)"},"comment":{"url":"https://api.github.com/repos/eslint/eslint/issues/comments/322287423","html_url":"https://github.com/eslint/eslint/issues/9059#issuecomment-322287423","issue_url":"https://api.github.com/repos/eslint/eslint/issues/9059","id":322287423,"user":{"login":"DonkeyAfterDark","id":9658655,"avatar_url":"https://avatars2.githubusercontent.com/u/9658655?v=4","gravatar_id":"","url":"https://api.github.com/users/DonkeyAfterDark","html_url":"https://github.com/DonkeyAfterDark","followers_url":"https://api.github.com/users/DonkeyAfterDark/followers","following_url":"https://api.github.com/users/DonkeyAfterDark/following{/other_user}","gists_url":"https://api.github.com/users/DonkeyAfterDark/gists{/gist_id}","starred_url":"https://api.github.com/users/DonkeyAfterDark/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DonkeyAfterDark/subscriptions","organizations_url":"https://api.github.com/users/DonkeyAfterDark/orgs","repos_url":"https://api.github.com/users/DonkeyAfterDark/repos","events_url":"https://api.github.com/users/DonkeyAfterDark/events{/privacy}","received_events_url":"https://api.github.com/users/DonkeyAfterDark/received_events","type":"User","site_admin":false},"created_at":"2017-08-14T19:36:31Z","updated_at":"2017-08-14T19:36:31Z","body":"Subtracting the string from the character count makes more sense."}}
{"actor":{"display_login":"DonkeyAfterDark"}}