{"action":"created","issue":{"url":"https://api.github.com/repos/eslint/eslint/issues/4512","labels_url":"https://api.github.com/repos/eslint/eslint/issues/4512/labels{/name}","comments_url":"https://api.github.com/repos/eslint/eslint/issues/4512/comments","events_url":"https://api.github.com/repos/eslint/eslint/issues/4512/events","html_url":"https://github.com/eslint/eslint/issues/4512","id":118229791,"number":4512,"title":"Merge variable rules into other sections of docs","user":{"login":"brettz9","id":20234,"avatar_url":"https://avatars.githubusercontent.com/u/20234?v=3","gravatar_id":"","url":"https://api.github.com/users/brettz9","html_url":"https://github.com/brettz9","followers_url":"https://api.github.com/users/brettz9/followers","following_url":"https://api.github.com/users/brettz9/following{/other_user}","gists_url":"https://api.github.com/users/brettz9/gists{/gist_id}","starred_url":"https://api.github.com/users/brettz9/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brettz9/subscriptions","organizations_url":"https://api.github.com/users/brettz9/orgs","repos_url":"https://api.github.com/users/brettz9/repos","events_url":"https://api.github.com/users/brettz9/events{/privacy}","received_events_url":"https://api.github.com/users/brettz9/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/eslint/eslint/labels/documentation","name":"documentation","color":"5319e7"},{"url":"https://api.github.com/repos/eslint/eslint/labels/proposal","name":"proposal","color":"fef2c0"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-11-22T00:33:11Z","updated_at":"2015-12-29T00:18:07Z","closed_at":null,"body":"Hi,\r\n\r\nI think your rules are now mostly well-categorized, but I would suggest a fully \"faceted browsing\" approach, i.e., more mutually exclusive categories, except where there is very good reason to separate a category out.\r\n\r\nAs it is currently, I think that you currently have two good, coherent high-level facets, namely, the degree of seriousness (Possible Errors, Best Practices, and Stylistic Issues), and Mode/Environment (Strict Mode, Node.js and Common.js, and ECMAScript 6).\r\n\r\nAlthough some of the environment rules could be merged into the degree of seriousness sections, I think the nature of these rules justifies separating them--though I would very much suggest grouping these sections all next to each other in the docs.\r\n\r\nWhile it would probably be ideal to put the rules into a table in order to allow for sorting by the type of rule (e.g., variables), degree of seriousness, etc, based on the current linear approach, I would suggest merging the variable rules into the other sections of the docs, so that projects can have just one place to look at for \"Possible Errors\" or \"Best Practices\". I think that is more critical than finding all of the variable-related items together.\r\n\r\nI would specifically suggest that `no-delete-var`, `no-undef`, and `no-shadow-restricted-names` be moved to \"Possible Errors\", that `no-catch-shadow`, `no-label-var`, `no-shadow`, and `no-use-before-define` to Best Practices, and the rest to \"Stylistic Issues\".\r\n\r\n<bountysource-plugin>\r\n---\r\nWant to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/28505022-merge-variable-rules-into-other-sections-of-docs?utm_campaign=plugin&utm_content=tracker%2F282608&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F282608&utm_medium=issues&utm_source=github).\r\n</bountysource-plugin>"},"comment":{"url":"https://api.github.com/repos/eslint/eslint/issues/comments/167682726","html_url":"https://github.com/eslint/eslint/issues/4512#issuecomment-167682726","issue_url":"https://api.github.com/repos/eslint/eslint/issues/4512","id":167682726,"user":{"login":"brettz9","id":20234,"avatar_url":"https://avatars.githubusercontent.com/u/20234?v=3","gravatar_id":"","url":"https://api.github.com/users/brettz9","html_url":"https://github.com/brettz9","followers_url":"https://api.github.com/users/brettz9/followers","following_url":"https://api.github.com/users/brettz9/following{/other_user}","gists_url":"https://api.github.com/users/brettz9/gists{/gist_id}","starred_url":"https://api.github.com/users/brettz9/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brettz9/subscriptions","organizations_url":"https://api.github.com/users/brettz9/orgs","repos_url":"https://api.github.com/users/brettz9/repos","events_url":"https://api.github.com/users/brettz9/events{/privacy}","received_events_url":"https://api.github.com/users/brettz9/received_events","type":"User","site_admin":false},"created_at":"2015-12-29T00:18:07Z","updated_at":"2015-12-29T00:18:07Z","body":"My PR #4555 was rejected as being too drastic (I put everything in a table so that it could potentially be sorted). I could make a PR which doesn't, but I really thought a table would be most convenient."}}