{"action":"created","issue":{"url":"https://api.github.com/repos/ReactiveX/rxjs/issues/1580","repository_url":"https://api.github.com/repos/ReactiveX/rxjs","labels_url":"https://api.github.com/repos/ReactiveX/rxjs/issues/1580/labels{/name}","comments_url":"https://api.github.com/repos/ReactiveX/rxjs/issues/1580/comments","events_url":"https://api.github.com/repos/ReactiveX/rxjs/issues/1580/events","html_url":"https://github.com/ReactiveX/rxjs/issues/1580","id":145905126,"number":1580,"title":"Installing beta.5 implicitly requires typings package installed","user":{"login":"kwonoj","id":1210596,"avatar_url":"https://avatars.githubusercontent.com/u/1210596?v=3","gravatar_id":"","url":"https://api.github.com/users/kwonoj","html_url":"https://github.com/kwonoj","followers_url":"https://api.github.com/users/kwonoj/followers","following_url":"https://api.github.com/users/kwonoj/following{/other_user}","gists_url":"https://api.github.com/users/kwonoj/gists{/gist_id}","starred_url":"https://api.github.com/users/kwonoj/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kwonoj/subscriptions","organizations_url":"https://api.github.com/users/kwonoj/orgs","repos_url":"https://api.github.com/users/kwonoj/repos","events_url":"https://api.github.com/users/kwonoj/events{/privacy}","received_events_url":"https://api.github.com/users/kwonoj/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2016-04-05T06:50:07Z","updated_at":"2016-04-15T00:06:19Z","closed_at":null,"body":"**RxJS version:**\r\nBeta.5\r\n\r\n**Code to reproduce:**\r\ncreate empty repo, do `npm install @reactivex/rxjs`\r\n\r\n**Actual behavior:**\r\n\r\n```sh\r\n> @reactivex/rxjs@5.0.0-beta.5 install Z:\\github\\test\\node_modules\\@reactivex\\rxjs\r\n> typings install\r\n\r\n'typings' is not recognized as an internal or external command,\r\noperable program or batch file.\r\n```\r\n\r\n**Additional information:**\r\n\r\nLatest change \r\nhttps://github.com/ReactiveX/rxjs/commit/41a1bbc43908f5e99c24b8c142bf6cd9d8aaa435#diff-b9cfc7f2cdf78a7f4b91a753d10865a2R51\r\n\r\noverrides behavior of `install`. Recommended practice is not to use `install` script, \r\n>Don't use install. Use a .gyp file for compilation, and prepublish for anything else. You should almost never have to explicitly set a preinstall or install script. If you are doing this, please consider if there is another option. The only valid use of install or preinstall scripts is for compilation which must be done on the target architecture. \r\n(https://docs.npmjs.com/misc/scripts#best-practices)\r\n\r\nand current script already utilizes `prepbulish` (https://github.com/ReactiveX/rxjs/blob/1b8600001f4226655afc54f3699ed3d0bfb0308d/package.json#L90) for lifecycle hooks.\r\n"},"comment":{"url":"https://api.github.com/repos/ReactiveX/rxjs/issues/comments/210211512","html_url":"https://github.com/ReactiveX/rxjs/issues/1580#issuecomment-210211512","issue_url":"https://api.github.com/repos/ReactiveX/rxjs/issues/1580","id":210211512,"user":{"login":"SomeKittens","id":1616464,"avatar_url":"https://avatars.githubusercontent.com/u/1616464?v=3","gravatar_id":"","url":"https://api.github.com/users/SomeKittens","html_url":"https://github.com/SomeKittens","followers_url":"https://api.github.com/users/SomeKittens/followers","following_url":"https://api.github.com/users/SomeKittens/following{/other_user}","gists_url":"https://api.github.com/users/SomeKittens/gists{/gist_id}","starred_url":"https://api.github.com/users/SomeKittens/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SomeKittens/subscriptions","organizations_url":"https://api.github.com/users/SomeKittens/orgs","repos_url":"https://api.github.com/users/SomeKittens/repos","events_url":"https://api.github.com/users/SomeKittens/events{/privacy}","received_events_url":"https://api.github.com/users/SomeKittens/received_events","type":"User","site_admin":false},"created_at":"2016-04-15T00:06:19Z","updated_at":"2016-04-15T00:06:19Z","body":"Huh, figured it was attached here because installing typings globally fixed the problem"}}