{"action":"created","issue":{"url":"https://api.github.com/repos/jhollingworth/marty/issues/212","labels_url":"https://api.github.com/repos/jhollingworth/marty/issues/212/labels{/name}","comments_url":"https://api.github.com/repos/jhollingworth/marty/issues/212/comments","events_url":"https://api.github.com/repos/jhollingworth/marty/issues/212/events","html_url":"https://github.com/jhollingworth/marty/pull/212","id":63966672,"number":212,"title":"Use let instead of var","user":{"login":"dariocravero","id":376268,"avatar_url":"https://avatars.githubusercontent.com/u/376268?v=3","gravatar_id":"","url":"https://api.github.com/users/dariocravero","html_url":"https://github.com/dariocravero","followers_url":"https://api.github.com/users/dariocravero/followers","following_url":"https://api.github.com/users/dariocravero/following{/other_user}","gists_url":"https://api.github.com/users/dariocravero/gists{/gist_id}","starred_url":"https://api.github.com/users/dariocravero/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dariocravero/subscriptions","organizations_url":"https://api.github.com/users/dariocravero/orgs","repos_url":"https://api.github.com/users/dariocravero/repos","events_url":"https://api.github.com/users/dariocravero/events{/privacy}","received_events_url":"https://api.github.com/users/dariocravero/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-03-24T10:48:32Z","updated_at":"2015-03-24T10:59:04Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/jhollingworth/marty/pulls/212","html_url":"https://github.com/jhollingworth/marty/pull/212","diff_url":"https://github.com/jhollingworth/marty/pull/212.diff","patch_url":"https://github.com/jhollingworth/marty/pull/212.patch"},"body":"As we're embracing ES6 more and more, I think that the first step would be to convert all of those `var`s to `let`s.\r\n\r\nI don't know how far we can go with tests though, are the fixtures passed through babel as well? Otherwise we might hit [some incompatibility on older browsers](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/let#Browser_compatibility)... \r\n\r\nAnother step forward would be to convert all the `require`s and `export`s to `import Module from 'module'` and `export default Module`, etc. I'm happy to go through it if you're ok with that?\r\n\r\nThoughts?"},"comment":{"url":"https://api.github.com/repos/jhollingworth/marty/issues/comments/85450303","html_url":"https://github.com/jhollingworth/marty/pull/212#issuecomment-85450303","issue_url":"https://api.github.com/repos/jhollingworth/marty/issues/212","id":85450303,"user":{"login":"dariocravero","id":376268,"avatar_url":"https://avatars.githubusercontent.com/u/376268?v=3","gravatar_id":"","url":"https://api.github.com/users/dariocravero","html_url":"https://github.com/dariocravero","followers_url":"https://api.github.com/users/dariocravero/followers","following_url":"https://api.github.com/users/dariocravero/following{/other_user}","gists_url":"https://api.github.com/users/dariocravero/gists{/gist_id}","starred_url":"https://api.github.com/users/dariocravero/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dariocravero/subscriptions","organizations_url":"https://api.github.com/users/dariocravero/orgs","repos_url":"https://api.github.com/users/dariocravero/repos","events_url":"https://api.github.com/users/dariocravero/events{/privacy}","received_events_url":"https://api.github.com/users/dariocravero/received_events","type":"User","site_admin":false},"created_at":"2015-03-24T10:59:04Z","updated_at":"2015-03-24T10:59:04Z","body":"Sounds good to me :) Great job on `0.9`!\r\n\r\nPersonally, I think that leaving node `0.10` behind would be a good thing. iojs is saving the day and helping us keep the pace at pushing the boundaries...\r\n\r\nWould that be possible in marty `0.10` (what an irony that the version numbers will match :))?\r\nThanks"}}