{"action":"created","issue":{"url":"https://api.github.com/repos/eslint/eslint/issues/6854","repository_url":"https://api.github.com/repos/eslint/eslint","labels_url":"https://api.github.com/repos/eslint/eslint/issues/6854/labels{/name}","comments_url":"https://api.github.com/repos/eslint/eslint/issues/6854/comments","events_url":"https://api.github.com/repos/eslint/eslint/issues/6854/events","html_url":"https://github.com/eslint/eslint/issues/6854","id":169707301,"number":6854,"title":"Consider trimming the plugins entries in .eslintrc","user":{"login":"jostrander","id":760989,"avatar_url":"https://avatars.githubusercontent.com/u/760989?v=3","gravatar_id":"","url":"https://api.github.com/users/jostrander","html_url":"https://github.com/jostrander","followers_url":"https://api.github.com/users/jostrander/followers","following_url":"https://api.github.com/users/jostrander/following{/other_user}","gists_url":"https://api.github.com/users/jostrander/gists{/gist_id}","starred_url":"https://api.github.com/users/jostrander/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jostrander/subscriptions","organizations_url":"https://api.github.com/users/jostrander/orgs","repos_url":"https://api.github.com/users/jostrander/repos","events_url":"https://api.github.com/users/jostrander/events{/privacy}","received_events_url":"https://api.github.com/users/jostrander/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/eslint/eslint/labels/accepted","name":"accepted","color":"0052cc"},{"url":"https://api.github.com/repos/eslint/eslint/labels/beginner","name":"beginner","color":"fef2c0"},{"url":"https://api.github.com/repos/eslint/eslint/labels/core","name":"core","color":"c7def8"},{"url":"https://api.github.com/repos/eslint/eslint/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":17,"created_at":"2016-08-05T22:20:51Z","updated_at":"2016-08-23T16:02:33Z","closed_at":null,"body":"**What version of ESLint are you using?**\r\n\r\n3.2.2\r\n\r\n**Please show your full configuration:**\r\n\r\n```\r\n{\r\n  \"plugins\": [ \"html \" ],\r\n  \"extends\": \"airbnb-base\"\r\n}\r\n```\r\n\r\n**What did you do? Please include the actual source code causing the issue.**\r\n\r\nCompletely user error: took me about 45 minutes to figure out the appended space in `.eslintrc` was causing the plugin to not be found. \r\n\r\nCan we consider have the plugins list configured to trim the entered values and/or give a warning that it had to be trimmed before trying to resolve them?\r\n\r\np.s. sorry for emitting the extraneous bug report template sections, they didn't really apply because it's simply user error on my end and the only relevant code is the `.eslintrc` file above."},"comment":{"url":"https://api.github.com/repos/eslint/eslint/issues/comments/241784000","html_url":"https://github.com/eslint/eslint/issues/6854#issuecomment-241784000","issue_url":"https://api.github.com/repos/eslint/eslint/issues/6854","id":241784000,"user":{"login":"platinumazure","id":284282,"avatar_url":"https://avatars.githubusercontent.com/u/284282?v=3","gravatar_id":"","url":"https://api.github.com/users/platinumazure","html_url":"https://github.com/platinumazure","followers_url":"https://api.github.com/users/platinumazure/followers","following_url":"https://api.github.com/users/platinumazure/following{/other_user}","gists_url":"https://api.github.com/users/platinumazure/gists{/gist_id}","starred_url":"https://api.github.com/users/platinumazure/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/platinumazure/subscriptions","organizations_url":"https://api.github.com/users/platinumazure/orgs","repos_url":"https://api.github.com/users/platinumazure/repos","events_url":"https://api.github.com/users/platinumazure/events{/privacy}","received_events_url":"https://api.github.com/users/platinumazure/received_events","type":"User","site_admin":false},"created_at":"2016-08-23T16:02:33Z","updated_at":"2016-08-23T16:02:33Z","body":"I raised a question about whether we should only validate whitespace on the issue, or if we should try to just check for a valid package name.\r\n\r\nThere's not a lot of information on npm but there's enough. https://docs.npmjs.com/files/package.json#name\r\n\r\n> Some rules:\r\n> \r\n> * The name must be less than or equal to 214 characters. This includes the scope for scoped packages.\r\n> * The name can't start with a dot or an underscore.\r\n> * New packages must not have uppercase letters in the name.\r\n> * The name ends up being part of a URL, an argument on the command line, and a folder name. Therefore, the name can't contain any non-URL-safe characters.\r\n\r\nUnfortunately, many of those are not very useful to us or would be hard to check accurately. Name starting with dot or underscore would only apply if the full package name is used in config. If we are confident that ESLint plugins were only supported after npm started forbidding uppercase letters, we could check that. And we could possibly check against non-URL-safe characters (which would probably take care of the whitespace problem).\r\n\r\nIf we've accepted this issue as a whitespace check only, then we can certainly merge PR #6960 and I can open a separate issue later."}}
{"actor":{"display_login":"platinumazure"}}