{"action":"created","issue":{"url":"https://api.github.com/repos/PolymerLabs/polyclean/issues/9","labels_url":"https://api.github.com/repos/PolymerLabs/polyclean/issues/9/labels{/name}","comments_url":"https://api.github.com/repos/PolymerLabs/polyclean/issues/9/comments","events_url":"https://api.github.com/repos/PolymerLabs/polyclean/issues/9/events","html_url":"https://github.com/PolymerLabs/polyclean/pull/9","id":124114961,"number":9,"title":"Fix trimming of spaces","user":{"login":"rubenstolk","id":692644,"avatar_url":"https://avatars.githubusercontent.com/u/692644?v=3","gravatar_id":"","url":"https://api.github.com/users/rubenstolk","html_url":"https://github.com/rubenstolk","followers_url":"https://api.github.com/users/rubenstolk/followers","following_url":"https://api.github.com/users/rubenstolk/following{/other_user}","gists_url":"https://api.github.com/users/rubenstolk/gists{/gist_id}","starred_url":"https://api.github.com/users/rubenstolk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rubenstolk/subscriptions","organizations_url":"https://api.github.com/users/rubenstolk/orgs","repos_url":"https://api.github.com/users/rubenstolk/repos","events_url":"https://api.github.com/users/rubenstolk/events{/privacy}","received_events_url":"https://api.github.com/users/rubenstolk/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"azakus","id":46725,"avatar_url":"https://avatars.githubusercontent.com/u/46725?v=3","gravatar_id":"","url":"https://api.github.com/users/azakus","html_url":"https://github.com/azakus","followers_url":"https://api.github.com/users/azakus/followers","following_url":"https://api.github.com/users/azakus/following{/other_user}","gists_url":"https://api.github.com/users/azakus/gists{/gist_id}","starred_url":"https://api.github.com/users/azakus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/azakus/subscriptions","organizations_url":"https://api.github.com/users/azakus/orgs","repos_url":"https://api.github.com/users/azakus/repos","events_url":"https://api.github.com/users/azakus/events{/privacy}","received_events_url":"https://api.github.com/users/azakus/received_events","type":"User","site_admin":false},"milestone":null,"comments":6,"created_at":"2015-12-28T19:51:30Z","updated_at":"2016-01-05T17:08:41Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/PolymerLabs/polyclean/pulls/9","html_url":"https://github.com/PolymerLabs/polyclean/pull/9","diff_url":"https://github.com/PolymerLabs/polyclean/pull/9.diff","patch_url":"https://github.com/PolymerLabs/polyclean/pull/9.patch"},"body":"Removing all occurrences of 2 spaces is dangerous. For example look at the following definition used in https://github.com/polymerelements/paper-styles:\r\n\r\n```css\r\n--shadow-elevation-16dp: {\r\n  box-shadow:\r\n    0 16px 24px 2px rgba(0, 0, 0, 0.14),\r\n    0  6px 30px 5px rgba(0, 0, 0, 0.12),\r\n    0  8px 10px -5px rgba(0, 0, 0, 0.4);\r\n};\r\n```\r\n\r\nFor indentation reasons the authors have used two spaces between `0` and `6px`/`8px`. Replacing two spaces or more with nothing would result in `06px` / `08px`.\r\n\r\nThis pull requests solves this by reducing two or more spaces to one (two or more will always have the same meaning as one) and by removing leading and trailing spaces afterwards.\r\n"},"comment":{"url":"https://api.github.com/repos/PolymerLabs/polyclean/issues/comments/169063846","html_url":"https://github.com/PolymerLabs/polyclean/pull/9#issuecomment-169063846","issue_url":"https://api.github.com/repos/PolymerLabs/polyclean/issues/9","id":169063846,"user":{"login":"ebidel","id":238208,"avatar_url":"https://avatars.githubusercontent.com/u/238208?v=3","gravatar_id":"","url":"https://api.github.com/users/ebidel","html_url":"https://github.com/ebidel","followers_url":"https://api.github.com/users/ebidel/followers","following_url":"https://api.github.com/users/ebidel/following{/other_user}","gists_url":"https://api.github.com/users/ebidel/gists{/gist_id}","starred_url":"https://api.github.com/users/ebidel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ebidel/subscriptions","organizations_url":"https://api.github.com/users/ebidel/orgs","repos_url":"https://api.github.com/users/ebidel/repos","events_url":"https://api.github.com/users/ebidel/events{/privacy}","received_events_url":"https://api.github.com/users/ebidel/received_events","type":"User","site_admin":false},"created_at":"2016-01-05T17:08:41Z","updated_at":"2016-01-05T17:08:41Z","body":"We won't make you setup tests for the repo in this PR :) I'll let @azakus have the final LGTM and merge b/c I can't update the npm module."}}