{"action":"created","issue":{"url":"https://api.github.com/repos/eslint/eslint/issues/3374","labels_url":"https://api.github.com/repos/eslint/eslint/issues/3374/labels{/name}","comments_url":"https://api.github.com/repos/eslint/eslint/issues/3374/comments","events_url":"https://api.github.com/repos/eslint/eslint/issues/3374/events","html_url":"https://github.com/eslint/eslint/pull/3374","id":100490885,"number":3374,"title":"Fix: Indent rule errors with array of objects (fixes #3329)","user":{"login":"BYK","id":126780,"avatar_url":"https://avatars.githubusercontent.com/u/126780?v=3","gravatar_id":"","url":"https://api.github.com/users/BYK","html_url":"https://github.com/BYK","followers_url":"https://api.github.com/users/BYK/followers","following_url":"https://api.github.com/users/BYK/following{/other_user}","gists_url":"https://api.github.com/users/BYK/gists{/gist_id}","starred_url":"https://api.github.com/users/BYK/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BYK/subscriptions","organizations_url":"https://api.github.com/users/BYK/orgs","repos_url":"https://api.github.com/users/BYK/repos","events_url":"https://api.github.com/users/BYK/events{/privacy}","received_events_url":"https://api.github.com/users/BYK/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-08-12T08:12:16Z","updated_at":"2015-08-12T14:27:12Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/eslint/eslint/pulls/3374","html_url":"https://github.com/eslint/eslint/pull/3374","diff_url":"https://github.com/eslint/eslint/pull/3374.diff","patch_url":"https://github.com/eslint/eslint/pull/3374.patch"},"body":"May not be the greatest fix for #3329 but it at least avoids double checking Array and Object expressions with conflicting rules so fixing the \"infinite\" or \"unfixable\" state that is caused by setting the `VariableDeclarator` option.\r\n\r\nNow I'd argue that this is also the expected behavior but I leave that decision to the reviewers and the author of the issue."},"comment":{"url":"https://api.github.com/repos/eslint/eslint/issues/comments/130321467","html_url":"https://github.com/eslint/eslint/pull/3374#issuecomment-130321467","issue_url":"https://api.github.com/repos/eslint/eslint/issues/3374","id":130321467,"user":{"login":"BYK","id":126780,"avatar_url":"https://avatars.githubusercontent.com/u/126780?v=3","gravatar_id":"","url":"https://api.github.com/users/BYK","html_url":"https://github.com/BYK","followers_url":"https://api.github.com/users/BYK/followers","following_url":"https://api.github.com/users/BYK/following{/other_user}","gists_url":"https://api.github.com/users/BYK/gists{/gist_id}","starred_url":"https://api.github.com/users/BYK/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BYK/subscriptions","organizations_url":"https://api.github.com/users/BYK/orgs","repos_url":"https://api.github.com/users/BYK/repos","events_url":"https://api.github.com/users/BYK/events{/privacy}","received_events_url":"https://api.github.com/users/BYK/received_events","type":"User","site_admin":false},"created_at":"2015-08-12T14:27:12Z","updated_at":"2015-08-12T14:27:12Z","body":">@BYK I am not sure either as we are entering a zone where its legal for some people and illegal for some. Now it is becoming difficult to say what would this rule check for what not (under objects and arrays).\r\n\r\nThis really just fixes the double-check/report behavior. It still checks everything it used to (thanks to tests, I'm confident in that). I just added a new test case and left all existing ones intact which reassures me that this is a completely backwards-compatible change."}}