{"action":"created","issue":{"url":"https://api.github.com/repos/brigade/scss-lint/issues/509","labels_url":"https://api.github.com/repos/brigade/scss-lint/issues/509/labels{/name}","comments_url":"https://api.github.com/repos/brigade/scss-lint/issues/509/comments","events_url":"https://api.github.com/repos/brigade/scss-lint/issues/509/events","html_url":"https://github.com/brigade/scss-lint/pull/509","id":92501867,"number":509,"title":"New linter: SpaceAroundOperator","user":{"login":"srawlins","id":103167,"avatar_url":"https://avatars.githubusercontent.com/u/103167?v=3","gravatar_id":"","url":"https://api.github.com/users/srawlins","html_url":"https://github.com/srawlins","followers_url":"https://api.github.com/users/srawlins/followers","following_url":"https://api.github.com/users/srawlins/following{/other_user}","gists_url":"https://api.github.com/users/srawlins/gists{/gist_id}","starred_url":"https://api.github.com/users/srawlins/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/srawlins/subscriptions","organizations_url":"https://api.github.com/users/srawlins/orgs","repos_url":"https://api.github.com/users/srawlins/repos","events_url":"https://api.github.com/users/srawlins/events{/privacy}","received_events_url":"https://api.github.com/users/srawlins/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/brigade/scss-lint/labels/enhancement","name":"enhancement","color":"bfe5bf"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":10,"created_at":"2015-07-01T22:49:36Z","updated_at":"2015-07-15T17:41:58Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/brigade/scss-lint/pulls/509","html_url":"https://github.com/brigade/scss-lint/pull/509","diff_url":"https://github.com/brigade/scss-lint/pull/509.diff","patch_url":"https://github.com/brigade/scss-lint/pull/509.patch"},"body":"Fixes #400 \r\n\r\nEssentially explained in the README change, and in [my comment on #400](https://github.com/brigade/scss-lint/issues/400#issuecomment-116768105).\r\n\r\nI think that `#all_matches` is a weak spot in this PR: it's not well unit-tested. However, it's not general purpose enough to go into `lib/scss-lint/utils.rb` (which also doesn't have it's own unit tests), and the specs revealed a few bugs that I fixed along the way. If this method should be implemented in some other way, I'm open."},"comment":{"url":"https://api.github.com/repos/brigade/scss-lint/issues/comments/121689672","html_url":"https://github.com/brigade/scss-lint/pull/509#issuecomment-121689672","issue_url":"https://api.github.com/repos/brigade/scss-lint/issues/509","id":121689672,"user":{"login":"srawlins","id":103167,"avatar_url":"https://avatars.githubusercontent.com/u/103167?v=3","gravatar_id":"","url":"https://api.github.com/users/srawlins","html_url":"https://github.com/srawlins","followers_url":"https://api.github.com/users/srawlins/followers","following_url":"https://api.github.com/users/srawlins/following{/other_user}","gists_url":"https://api.github.com/users/srawlins/gists{/gist_id}","starred_url":"https://api.github.com/users/srawlins/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/srawlins/subscriptions","organizations_url":"https://api.github.com/users/srawlins/orgs","repos_url":"https://api.github.com/users/srawlins/repos","events_url":"https://api.github.com/users/srawlins/events{/privacy}","received_events_url":"https://api.github.com/users/srawlins/received_events","type":"User","site_admin":false},"created_at":"2015-07-15T17:41:58Z","updated_at":"2015-07-15T17:41:58Z","body":"Oops, looks like one spec now fails on Sass 3.4.16 ([Travis](https://travis-ci.org/brigade/scss-lint/builds/71117379)), but this spec does not fail for me on Sass 3.4.15. From Sass's [CHANGELOG](https://github.com/sass/sass/blob/stable/doc-src/SASS_CHANGELOG.md#3416-10-july-2015):\r\n\r\n> Parse a minus operator with no surrounding whitespace (e.g. 20px-10px) the same regardless of whether it's being used in a static property.\r\n\r\nSo newly in 3.4.16, `margin: 2em-1em;` has an operation in it. Unfortunately this _should_ be a lint in 3.4.16, and _should not_ be a lint <= 3.4.15, so I've just commented out the example for now."}}