{"action":"created","issue":{"url":"https://api.github.com/repos/KingScooty/creativenightly.com/issues/36","repository_url":"https://api.github.com/repos/KingScooty/creativenightly.com","labels_url":"https://api.github.com/repos/KingScooty/creativenightly.com/issues/36/labels{/name}","comments_url":"https://api.github.com/repos/KingScooty/creativenightly.com/issues/36/comments","events_url":"https://api.github.com/repos/KingScooty/creativenightly.com/issues/36/events","html_url":"https://github.com/KingScooty/creativenightly.com/pull/36","id":133665812,"number":36,"title":"Use less lenient selector list config?","user":{"login":"jeddy3","id":808227,"avatar_url":"https://avatars.githubusercontent.com/u/808227?v=3","gravatar_id":"","url":"https://api.github.com/users/jeddy3","html_url":"https://github.com/jeddy3","followers_url":"https://api.github.com/users/jeddy3/followers","following_url":"https://api.github.com/users/jeddy3/following{/other_user}","gists_url":"https://api.github.com/users/jeddy3/gists{/gist_id}","starred_url":"https://api.github.com/users/jeddy3/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jeddy3/subscriptions","organizations_url":"https://api.github.com/users/jeddy3/orgs","repos_url":"https://api.github.com/users/jeddy3/repos","events_url":"https://api.github.com/users/jeddy3/events{/privacy}","received_events_url":"https://api.github.com/users/jeddy3/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2016-02-15T09:37:37Z","updated_at":"2016-02-15T11:49:07Z","closed_at":"2016-02-15T10:48:24Z","pull_request":{"url":"https://api.github.com/repos/KingScooty/creativenightly.com/pulls/36","html_url":"https://github.com/KingScooty/creativenightly.com/pull/36","diff_url":"https://github.com/KingScooty/creativenightly.com/pull/36.diff","patch_url":"https://github.com/KingScooty/creativenightly.com/pull/36.patch"},"body":"@KingScooty Hello.\r\n\r\nThank you for writing such a fantastic article about stylelint! It’s hands-down the best one I’ve seen :)\r\n\r\nI’m going to tweet it from the @ stylelint account, but before I do I just wanted to run something by you as there’s one tiny thing in your example config that might confuse first-time users.\r\n\r\nIt’s the `”always-multi-line”` option for the `”selector-list-comma-newline-after”` rule. I’m wondering if the `”always-multi-line”` option is going to be a bit confusing for first-time users (who might copy-paste your config), as it’s very lenient and would allow all of the following:\r\n\r\n```css\r\n/* single line selector lists, and in any style e.g. */\r\na, b { }\r\n\r\n/* … and */\r\na ,b { }\r\n\r\n/* … and */\r\na,b { }\r\n\r\n/* as well as multiline selector lists in a variety of styles */\r\na, \r\nb {}\r\n\r\n/* …and */\r\na ,\r\nb {}\r\n\r\n/* …and */\r\na\r\n,\r\nb {}\r\n\r\n```\r\n\r\nIf you’re looking to enforce just this style (am I right in thinking this is what you’re after?):\r\n\r\n```css\r\na,\r\nb {}\r\n```\r\n\r\nThen your config will need to be:\r\n\r\n```json\r\n{\r\n  \"selector-list-comma-space-before\": \"never\",\r\n  \"selector-list-comma-newline-after\": \"always\"\r\n}\r\n```\r\n\r\nJust like you’ve done with the `declaration-colon-space-*` and `media-feature-colon-space-*` rules, all the `*-after` and `*-before` rules [tend to go together](https://github.com/stylelint/stylelint/blob/master/docs/user-guide/configuration.md#rules-work-together).\r\n\r\nThe `*-multi-line` and `*-single-line` options are a bit tricky, but allow a user to enforce [all kinds of crazy](https://github.com/stylelint/stylelint/blob/master/docs/user-guide/configuration.md#example-d) styles. Do you think it’s worth skipping any `*-multi-line*` and `*-single-line` options in the article to keep things nice and simple for new starters?\r\n\r\nThanks again for the superb article and I’m glad you’re finding stylelint useful :)\r\n\r\nP.S. I thought I’d knock this question up as a Pull Request, with the change to the config, as to save you a moment of time or two. It’s not possible do the same for the gulp gists though. Feel free to either merge or, if I’ve totally gotten the wrong of the stick regarding the config, close this Pull Request. I’ll be tweeting the article either way, if that’s OK? :)\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/KingScooty/creativenightly.com/issues/comments/184178521","html_url":"https://github.com/KingScooty/creativenightly.com/pull/36#issuecomment-184178521","issue_url":"https://api.github.com/repos/KingScooty/creativenightly.com/issues/36","id":184178521,"user":{"login":"jeddy3","id":808227,"avatar_url":"https://avatars.githubusercontent.com/u/808227?v=3","gravatar_id":"","url":"https://api.github.com/users/jeddy3","html_url":"https://github.com/jeddy3","followers_url":"https://api.github.com/users/jeddy3/followers","following_url":"https://api.github.com/users/jeddy3/following{/other_user}","gists_url":"https://api.github.com/users/jeddy3/gists{/gist_id}","starred_url":"https://api.github.com/users/jeddy3/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jeddy3/subscriptions","organizations_url":"https://api.github.com/users/jeddy3/orgs","repos_url":"https://api.github.com/users/jeddy3/repos","events_url":"https://api.github.com/users/jeddy3/events{/privacy}","received_events_url":"https://api.github.com/users/jeddy3/received_events","type":"User","site_admin":false},"created_at":"2016-02-15T11:49:07Z","updated_at":"2016-02-15T11:49:07Z","body":"> So i'm going to merge this in and update the gists :)\r\n\r\nExcellent, thanks.\r\n\r\n> If you have any other suggestions, by all means let me know! :)\r\n\r\nEverything else is great! It's a fantastic primer for new users. Thanks again for writing it :)"}}