{"action":"created","issue":{"url":"https://api.github.com/repos/jenius/accord/issues/87","labels_url":"https://api.github.com/repos/jenius/accord/issues/87/labels{/name}","comments_url":"https://api.github.com/repos/jenius/accord/issues/87/comments","events_url":"https://api.github.com/repos/jenius/accord/issues/87/events","html_url":"https://github.com/jenius/accord/pull/87","id":61891867,"number":87,"title":"Add node-sass 1.x 2.x and 3.x compatibility in scss adapter. Fixes #86","user":{"login":"Munter","id":331790,"avatar_url":"https://avatars.githubusercontent.com/u/331790?v=3","gravatar_id":"","url":"https://api.github.com/users/Munter","html_url":"https://github.com/Munter","followers_url":"https://api.github.com/users/Munter/followers","following_url":"https://api.github.com/users/Munter/following{/other_user}","gists_url":"https://api.github.com/users/Munter/gists{/gist_id}","starred_url":"https://api.github.com/users/Munter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Munter/subscriptions","organizations_url":"https://api.github.com/users/Munter/orgs","repos_url":"https://api.github.com/users/Munter/repos","events_url":"https://api.github.com/users/Munter/events{/privacy}","received_events_url":"https://api.github.com/users/Munter/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-03-15T20:11:52Z","updated_at":"2015-03-15T20:22:12Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/jenius/accord/pulls/87","html_url":"https://github.com/jenius/accord/pull/87","diff_url":"https://github.com/jenius/accord/pull/87.diff","patch_url":"https://github.com/jenius/accord/pull/87.patch"},"body":"This makes the scss adapter compatible between node-sass 1.x 2.x and 3.x.\r\n\r\nTested with 3.0.0-pre, 2.0.1 and 1.2.3\r\n\r\nI couldn't get 1.2.3 sourcemaps to work, but this is at least better than the state it was in before"},"comment":{"url":"https://api.github.com/repos/jenius/accord/issues/comments/81224742","html_url":"https://github.com/jenius/accord/pull/87#issuecomment-81224742","issue_url":"https://api.github.com/repos/jenius/accord/issues/87","id":81224742,"user":{"login":"jenius","id":556932,"avatar_url":"https://avatars.githubusercontent.com/u/556932?v=3","gravatar_id":"","url":"https://api.github.com/users/jenius","html_url":"https://github.com/jenius","followers_url":"https://api.github.com/users/jenius/followers","following_url":"https://api.github.com/users/jenius/following{/other_user}","gists_url":"https://api.github.com/users/jenius/gists{/gist_id}","starred_url":"https://api.github.com/users/jenius/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jenius/subscriptions","organizations_url":"https://api.github.com/users/jenius/orgs","repos_url":"https://api.github.com/users/jenius/repos","events_url":"https://api.github.com/users/jenius/events{/privacy}","received_events_url":"https://api.github.com/users/jenius/received_events","type":"User","site_admin":false},"created_at":"2015-03-15T20:22:12Z","updated_at":"2015-03-15T20:22:12Z","body":"Ok so this seems like it would work for an immediate relief patch, but I'm not as confident in how this would go as a long term solution. How are we going to test different versions? For other libraries for which we add backwards compatibility, how are we going to handle more subtle changes where you'd actually need to know the version to be able to detect it? How many versions back should we support for any given library?\r\n\r\nBefore merging something like this and going down that rabbit hole, I'd like to hammer out the answers to questions like these so we are confident down the line in additions like this, rather than letting each adapter slowly turn into spaghetti code because of it."}}