{"action":"created","issue":{"url":"https://api.github.com/repos/stylelint/stylelint/issues/177","labels_url":"https://api.github.com/repos/stylelint/stylelint/issues/177/labels{/name}","comments_url":"https://api.github.com/repos/stylelint/stylelint/issues/177/comments","events_url":"https://api.github.com/repos/stylelint/stylelint/issues/177/events","html_url":"https://github.com/stylelint/stylelint/pull/177","id":86142675,"number":177,"title":"Extra tests for semicolon-newline-after","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":[{"url":"https://api.github.com/repos/stylelint/stylelint/labels/PR:%20needs%20revision","name":"PR: needs revision","color":"B20C00"},{"url":"https://api.github.com/repos/stylelint/stylelint/labels/status:%20help%20wanted","name":"status: help wanted","color":"fbca04"}],"state":"closed","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/stylelint/stylelint/milestones/1","html_url":"https://github.com/stylelint/stylelint/milestones/0.1.0","labels_url":"https://api.github.com/repos/stylelint/stylelint/milestones/1/labels","id":1104657,"number":1,"title":"0.1.0","description":"","creator":{"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},"open_issues":2,"closed_issues":26,"state":"open","created_at":"2015-05-11T12:12:37Z","updated_at":"2015-06-09T18:39:06Z","due_on":null,"closed_at":null},"comments":3,"created_at":"2015-06-08T11:45:44Z","updated_at":"2015-06-09T21:29:59Z","closed_at":"2015-06-09T18:39:06Z","pull_request":{"url":"https://api.github.com/repos/stylelint/stylelint/pulls/177","html_url":"https://github.com/stylelint/stylelint/pull/177","diff_url":"https://github.com/stylelint/stylelint/pull/177.diff","patch_url":"https://github.com/stylelint/stylelint/pull/177.patch"},"body":"Just adding the remaining rules to my test bed and I'm not sure if `rule-semicolon-newline-after` is behaving as expected. Given\r\n\r\n```css\r\na {\r\n  color: pink;\r\n  top: 0;\r\n}\r\n```\r\n\r\nAnd\r\n\r\n```js\r\nrule-semicolon-newline-after: [2, \"always\"]\r\n```\r\n\r\nA warning is flagged for the semicolon after `pink`. I think it's because of the indentation. I've added a couple more tests to the rule. To replicate this. I think I've done it right.\r\n\r\nI figured I'd push it up here so you could take a look before I delve into the inner workings of the rule. Is this a bug or have I misunderstood the expected behaviour of this rule?\r\n\r\nThanks!"},"comment":{"url":"https://api.github.com/repos/stylelint/stylelint/issues/comments/110509141","html_url":"https://github.com/stylelint/stylelint/pull/177#issuecomment-110509141","issue_url":"https://api.github.com/repos/stylelint/stylelint/issues/177","id":110509141,"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":"2015-06-09T21:29:59Z","updated_at":"2015-06-09T21:29:59Z","body":"> I like the strategy of adding failing tests. Thanks.\r\n\r\nI'm glad you like it!\r\n\r\nThanks for fixing. Master is working fine against https://github.com/stylelint/stylelint.github.io/ now."}}