{"action":"created","issue":{"url":"https://api.github.com/repos/tmuxinator/tmuxinator/issues/386","repository_url":"https://api.github.com/repos/tmuxinator/tmuxinator","labels_url":"https://api.github.com/repos/tmuxinator/tmuxinator/issues/386/labels{/name}","comments_url":"https://api.github.com/repos/tmuxinator/tmuxinator/issues/386/comments","events_url":"https://api.github.com/repos/tmuxinator/tmuxinator/issues/386/events","html_url":"https://github.com/tmuxinator/tmuxinator/pull/386","id":143344086,"number":386,"title":"Completions: test if tmuxinator is actually on $PATH","user":{"login":"wildefyr","id":8454096,"avatar_url":"https://avatars.githubusercontent.com/u/8454096?v=3","gravatar_id":"","url":"https://api.github.com/users/wildefyr","html_url":"https://github.com/wildefyr","followers_url":"https://api.github.com/users/wildefyr/followers","following_url":"https://api.github.com/users/wildefyr/following{/other_user}","gists_url":"https://api.github.com/users/wildefyr/gists{/gist_id}","starred_url":"https://api.github.com/users/wildefyr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wildefyr/subscriptions","organizations_url":"https://api.github.com/users/wildefyr/orgs","repos_url":"https://api.github.com/users/wildefyr/repos","events_url":"https://api.github.com/users/wildefyr/events{/privacy}","received_events_url":"https://api.github.com/users/wildefyr/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":11,"created_at":"2016-03-24T19:57:15Z","updated_at":"2016-04-01T23:07:07Z","closed_at":"2016-03-24T20:24:13Z","pull_request":{"url":"https://api.github.com/repos/tmuxinator/tmuxinator/pulls/386","html_url":"https://github.com/tmuxinator/tmuxinator/pull/386","diff_url":"https://github.com/tmuxinator/tmuxinator/pull/386.diff","patch_url":"https://github.com/tmuxinator/tmuxinator/pull/386.patch"},"body":"I'm sure others carry their shell environment around as well on to machines where they don't want to install tmuxinator. As such trying to source the completion script results in the annoying message of 'tmuxinator: not found'. If I want tmuxinator, I'll install it if I need it."},"comment":{"url":"https://api.github.com/repos/tmuxinator/tmuxinator/issues/comments/204603066","html_url":"https://github.com/tmuxinator/tmuxinator/pull/386#issuecomment-204603066","issue_url":"https://api.github.com/repos/tmuxinator/tmuxinator/issues/386","id":204603066,"user":{"login":"cinaeco","id":515654,"avatar_url":"https://avatars.githubusercontent.com/u/515654?v=3","gravatar_id":"","url":"https://api.github.com/users/cinaeco","html_url":"https://github.com/cinaeco","followers_url":"https://api.github.com/users/cinaeco/followers","following_url":"https://api.github.com/users/cinaeco/following{/other_user}","gists_url":"https://api.github.com/users/cinaeco/gists{/gist_id}","starred_url":"https://api.github.com/users/cinaeco/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cinaeco/subscriptions","organizations_url":"https://api.github.com/users/cinaeco/orgs","repos_url":"https://api.github.com/users/cinaeco/repos","events_url":"https://api.github.com/users/cinaeco/events{/privacy}","received_events_url":"https://api.github.com/users/cinaeco/received_events","type":"User","site_admin":false},"created_at":"2016-04-01T23:07:07Z","updated_at":"2016-04-01T23:07:07Z","body":"It does the same on two bashes: Apple's built-in 3.2+ and Centos's built-in 4.2+... I don't think this is an issue of posix compliance. `type` is fine. I personally like `command -v` better, which is also posix compliance. The 2 issues with this pull request I see are:\r\n\r\n1. It seems stderr is wrongly redirected - it was intended to be sent to /dev/null, but isn't. That's easy to fix, but...\r\n2. A `tmuxinator` existence check is just plain unnecessary, because `complete` works fine without it. At least, I don't see any cons. Not having the existence check makes tmuxinator.bash More Flexible because it means `tmuxinator` actually doesn't have to be detectable at time of sourcing in order for completion to work."}}