{"action":"created","issue":{"url":"https://api.github.com/repos/eslint/eslint/issues/7473","repository_url":"https://api.github.com/repos/eslint/eslint","labels_url":"https://api.github.com/repos/eslint/eslint/issues/7473/labels{/name}","comments_url":"https://api.github.com/repos/eslint/eslint/issues/7473/comments","events_url":"https://api.github.com/repos/eslint/eslint/issues/7473/events","html_url":"https://github.com/eslint/eslint/issues/7473","id":186036938,"number":7473,"title":"Version 3.9.0 broke multiline array indentation","user":{"login":"cyrus-and","id":1660263,"avatar_url":"https://avatars.githubusercontent.com/u/1660263?v=3","gravatar_id":"","url":"https://api.github.com/users/cyrus-and","html_url":"https://github.com/cyrus-and","followers_url":"https://api.github.com/users/cyrus-and/followers","following_url":"https://api.github.com/users/cyrus-and/following{/other_user}","gists_url":"https://api.github.com/users/cyrus-and/gists{/gist_id}","starred_url":"https://api.github.com/users/cyrus-and/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cyrus-and/subscriptions","organizations_url":"https://api.github.com/users/cyrus-and/orgs","repos_url":"https://api.github.com/users/cyrus-and/repos","events_url":"https://api.github.com/users/cyrus-and/events{/privacy}","received_events_url":"https://api.github.com/users/cyrus-and/received_events","type":"User","site_admin":false},"labels":[{"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":7,"created_at":"2016-10-28T22:35:40Z","updated_at":"2016-11-03T18:39:13Z","closed_at":null,"body":"<!--\nThis template is for bug reports. If you are reporting a bug, please continue on. If you are here for another reason, please see below:\n\n1. To propose a new rule: http://eslint.org/docs/developer-guide/contributing/new-rules\n2. To request a change: http://eslint.org/docs/developer-guide/contributing/changes\n3. If you have any questions, please stop by our chatroom: https://gitter.im/eslint/eslint\n\nNote that leaving sections blank will make it difficult for us to troubleshoot and we may have to close the issue.\n-->\n\n**Tell us about your environment**\n- **ESLint Version:** v3.9.0\n- **Node Version:** v7.0.0-pre\n- **npm Version:** 3.10.8\n\n**What parser (default, Babel-ESLint, etc.) are you using?** default\n\n**Please show your full configuration:**\n\n<!-- Paste your configuration below: -->\n\n```\n{\n    \"rules\": {\n        \"indent\": [\n            \"error\",\n            4\n        ]\n    }\n}\n```\n\n**What did you do? Please include the actual source code causing the issue.**\n\n<!-- Paste the source code below: -->\n\n``` js\nvar array = [1,\n             2,\n             3];\n```\n\n**What did you expect to happen?**\n\nThis was considered valid before this release. It would be great to have the `first` option for arrays too.\n\n**What actually happened? Please include the actual, raw output from ESLint.**\n\n```\n2:14  error  Expected indentation of 4 spaces but found 13  indent\n3:14  error  Expected indentation of 4 spaces but found 13  indent\n```\n"},"comment":{"url":"https://api.github.com/repos/eslint/eslint/issues/comments/258235774","html_url":"https://github.com/eslint/eslint/issues/7473#issuecomment-258235774","issue_url":"https://api.github.com/repos/eslint/eslint/issues/7473","id":258235774,"user":{"login":"nunocastromartins","id":8314616,"avatar_url":"https://avatars.githubusercontent.com/u/8314616?v=3","gravatar_id":"","url":"https://api.github.com/users/nunocastromartins","html_url":"https://github.com/nunocastromartins","followers_url":"https://api.github.com/users/nunocastromartins/followers","following_url":"https://api.github.com/users/nunocastromartins/following{/other_user}","gists_url":"https://api.github.com/users/nunocastromartins/gists{/gist_id}","starred_url":"https://api.github.com/users/nunocastromartins/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nunocastromartins/subscriptions","organizations_url":"https://api.github.com/users/nunocastromartins/orgs","repos_url":"https://api.github.com/users/nunocastromartins/repos","events_url":"https://api.github.com/users/nunocastromartins/events{/privacy}","received_events_url":"https://api.github.com/users/nunocastromartins/received_events","type":"User","site_admin":false},"created_at":"2016-11-03T18:39:13Z","updated_at":"2016-11-03T18:39:13Z","body":"**Note:** version 3.9.0 also broke multi-line object indentation.\r\n\r\n### My two cents on this issue:\r\nIndentation checking is disabled by default on function parameters and call expression arguments. Why must it be enabled by default on arrays/objects? The change _broke_ a lot of my code (where _broke_ reads as: ESLint now complains about it).\r\n\r\nI agree that the option to indent array elements and object properties should be added, possibly with a `first` option, but with consistency in mind, it should probably be disabled by default."}}
{"actor":{"display_login":"nunocastromartins"}}