{"action":"created","issue":{"url":"https://api.github.com/repos/atom/ci/issues/43","repository_url":"https://api.github.com/repos/atom/ci","labels_url":"https://api.github.com/repos/atom/ci/issues/43/labels{/name}","comments_url":"https://api.github.com/repos/atom/ci/issues/43/comments","events_url":"https://api.github.com/repos/atom/ci/issues/43/events","html_url":"https://github.com/atom/ci/pull/43","id":163997394,"number":43,"title":"Don't modify PATH on OS X","user":{"login":"Arcanemagus","id":427137,"avatar_url":"https://avatars.githubusercontent.com/u/427137?v=3","gravatar_id":"","url":"https://api.github.com/users/Arcanemagus","html_url":"https://github.com/Arcanemagus","followers_url":"https://api.github.com/users/Arcanemagus/followers","following_url":"https://api.github.com/users/Arcanemagus/following{/other_user}","gists_url":"https://api.github.com/users/Arcanemagus/gists{/gist_id}","starred_url":"https://api.github.com/users/Arcanemagus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Arcanemagus/subscriptions","organizations_url":"https://api.github.com/users/Arcanemagus/orgs","repos_url":"https://api.github.com/users/Arcanemagus/repos","events_url":"https://api.github.com/users/Arcanemagus/events{/privacy}","received_events_url":"https://api.github.com/users/Arcanemagus/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"thomasjo","id":3622,"avatar_url":"https://avatars.githubusercontent.com/u/3622?v=3","gravatar_id":"","url":"https://api.github.com/users/thomasjo","html_url":"https://github.com/thomasjo","followers_url":"https://api.github.com/users/thomasjo/followers","following_url":"https://api.github.com/users/thomasjo/following{/other_user}","gists_url":"https://api.github.com/users/thomasjo/gists{/gist_id}","starred_url":"https://api.github.com/users/thomasjo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thomasjo/subscriptions","organizations_url":"https://api.github.com/users/thomasjo/orgs","repos_url":"https://api.github.com/users/thomasjo/repos","events_url":"https://api.github.com/users/thomasjo/events{/privacy}","received_events_url":"https://api.github.com/users/thomasjo/received_events","type":"User","site_admin":false},"assignees":[{"login":"thomasjo","id":3622,"avatar_url":"https://avatars.githubusercontent.com/u/3622?v=3","gravatar_id":"","url":"https://api.github.com/users/thomasjo","html_url":"https://github.com/thomasjo","followers_url":"https://api.github.com/users/thomasjo/followers","following_url":"https://api.github.com/users/thomasjo/following{/other_user}","gists_url":"https://api.github.com/users/thomasjo/gists{/gist_id}","starred_url":"https://api.github.com/users/thomasjo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thomasjo/subscriptions","organizations_url":"https://api.github.com/users/thomasjo/orgs","repos_url":"https://api.github.com/users/thomasjo/repos","events_url":"https://api.github.com/users/thomasjo/events{/privacy}","received_events_url":"https://api.github.com/users/thomasjo/received_events","type":"User","site_admin":false}],"milestone":null,"comments":3,"created_at":"2016-07-06T04:45:13Z","updated_at":"2016-07-07T08:22:53Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/atom/ci/pulls/43","html_url":"https://github.com/atom/ci/pull/43","diff_url":"https://github.com/atom/ci/pull/43.diff","patch_url":"https://github.com/atom/ci/pull/43.patch"},"body":"There is a node executable in the apm/bin directory, which overrides any version of Node.js installed on the system since the PATH is being updated to put this path at the front. As the Linux side of the script does not do this, remove the modification of the PATH to stop breaking Node.js.\n\nYou can see a [simplified version](https://github.com/AtomLinter/linter-htmlhint/blob/8ff7d7e73842f2e41503f4d8cea1c5c10713a9eb/x.sh) of the script being run [here](https://travis-ci.org/AtomLinter/linter-htmlhint/builds/142656364). This shows that Node.js is v6.2.2 at the start of the script, but after the `PATH` modification it shows Node.js at v0.10.40.\n\nI'm not sure why this modification was there in the first place though, @joefitzgerald as you are the original author of that do you know if this is necessary?"},"comment":{"url":"https://api.github.com/repos/atom/ci/issues/comments/231014010","html_url":"https://github.com/atom/ci/pull/43#issuecomment-231014010","issue_url":"https://api.github.com/repos/atom/ci/issues/43","id":231014010,"user":{"login":"thomasjo","id":3622,"avatar_url":"https://avatars.githubusercontent.com/u/3622?v=3","gravatar_id":"","url":"https://api.github.com/users/thomasjo","html_url":"https://github.com/thomasjo","followers_url":"https://api.github.com/users/thomasjo/followers","following_url":"https://api.github.com/users/thomasjo/following{/other_user}","gists_url":"https://api.github.com/users/thomasjo/gists{/gist_id}","starred_url":"https://api.github.com/users/thomasjo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thomasjo/subscriptions","organizations_url":"https://api.github.com/users/thomasjo/orgs","repos_url":"https://api.github.com/users/thomasjo/repos","events_url":"https://api.github.com/users/thomasjo/events{/privacy}","received_events_url":"https://api.github.com/users/thomasjo/received_events","type":"User","site_admin":false},"created_at":"2016-07-07T08:22:53Z","updated_at":"2016-07-07T08:22:53Z","body":"I'm gonna merge this. If it breaks stuff, blame me 🔫 "}}
{"actor":{"display_login":"thomasjo"}}