{"action":"created","issue":{"url":"https://api.github.com/repos/DefinitelyTyped/DefinitelyTyped/issues/28069","repository_url":"https://api.github.com/repos/DefinitelyTyped/DefinitelyTyped","labels_url":"https://api.github.com/repos/DefinitelyTyped/DefinitelyTyped/issues/28069/labels{/name}","comments_url":"https://api.github.com/repos/DefinitelyTyped/DefinitelyTyped/issues/28069/comments","events_url":"https://api.github.com/repos/DefinitelyTyped/DefinitelyTyped/issues/28069/events","html_url":"https://github.com/DefinitelyTyped/DefinitelyTyped/issues/28069","id":349934974,"node_id":"MDU6SXNzdWUzNDk5MzQ5NzQ=","number":28069,"title":"[@types/node] child_process StdioOptions does not allow \"inherit\" in array form anymore.","user":{"login":"mgroenhoff","id":22659163,"node_id":"MDQ6VXNlcjIyNjU5MTYz","avatar_url":"https://avatars0.githubusercontent.com/u/22659163?v=4","gravatar_id":"","url":"https://api.github.com/users/mgroenhoff","html_url":"https://github.com/mgroenhoff","followers_url":"https://api.github.com/users/mgroenhoff/followers","following_url":"https://api.github.com/users/mgroenhoff/following{/other_user}","gists_url":"https://api.github.com/users/mgroenhoff/gists{/gist_id}","starred_url":"https://api.github.com/users/mgroenhoff/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mgroenhoff/subscriptions","organizations_url":"https://api.github.com/users/mgroenhoff/orgs","repos_url":"https://api.github.com/users/mgroenhoff/repos","events_url":"https://api.github.com/users/mgroenhoff/events{/privacy}","received_events_url":"https://api.github.com/users/mgroenhoff/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-08-13T08:34:34Z","updated_at":"2018-08-13T10:17:30Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"- [x] I tried using the `@types/node` package and had problems.\r\n- [x] I tried using the latest stable version of tsc. https://www.npmjs.com/package/typescript\r\n- [ ] I have a question that is inappropriate for [StackOverflow](https://stackoverflow.com/).  (Please ask any appropriate questions there).\r\n- [x] [Mention](https://github.com/blog/821-mention-somebody-they-re-notified) the authors (see `Definitions by:` in `index.d.ts`) so they can respond.\r\n  - Authors: @thw0rted @andy-ms \r\n\r\nThe perfectly valid value `[\"inherit\", \"pipe\", \"pipe\", \"ipc\"]` for option `stdio` is no longer accepted and fails to compile.\r\n\r\nCause: #28025\r\n\r\nCan we please stop making backwards-incompatible changes in patch versions please?"},"comment":{"url":"https://api.github.com/repos/DefinitelyTyped/DefinitelyTyped/issues/comments/412472322","html_url":"https://github.com/DefinitelyTyped/DefinitelyTyped/issues/28069#issuecomment-412472322","issue_url":"https://api.github.com/repos/DefinitelyTyped/DefinitelyTyped/issues/28069","id":412472322,"node_id":"MDEyOklzc3VlQ29tbWVudDQxMjQ3MjMyMg==","user":{"login":"thw0rted","id":943160,"node_id":"MDQ6VXNlcjk0MzE2MA==","avatar_url":"https://avatars3.githubusercontent.com/u/943160?v=4","gravatar_id":"","url":"https://api.github.com/users/thw0rted","html_url":"https://github.com/thw0rted","followers_url":"https://api.github.com/users/thw0rted/followers","following_url":"https://api.github.com/users/thw0rted/following{/other_user}","gists_url":"https://api.github.com/users/thw0rted/gists{/gist_id}","starred_url":"https://api.github.com/users/thw0rted/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thw0rted/subscriptions","organizations_url":"https://api.github.com/users/thw0rted/orgs","repos_url":"https://api.github.com/users/thw0rted/repos","events_url":"https://api.github.com/users/thw0rted/events{/privacy}","received_events_url":"https://api.github.com/users/thw0rted/received_events","type":"User","site_admin":false},"created_at":"2018-08-13T10:17:30Z","updated_at":"2018-08-13T10:17:30Z","author_association":"CONTRIBUTOR","body":"@mgroenhoff Sorry, this is a(nother) oversight in the Node docs.  As you point out, \"inherit\" is a valid string to pass in the array-syntax but the docs [say it is not](https://nodejs.org/api/child_process.html#child_process_options_stdio), even though it's used in an example [further down the same page](https://nodejs.org/api/child_process.html#child_process_subprocess_kill_signal).  I'll get together a PR for both here and the Node docs (again!).\r\n\r\nAs for not making breaking changes in patch versions, I'm honestly not sure what else to do.  Last time I submitted a similar change (specifying the actual type of a parameter that was previously `any`), people's code broke and they complained, but as far as I can tell DT really doesn't have a way to schedule or wall off changes to prevent that -- DT can't follow semver, by design.   If you know of a process we could follow to prevent patch-version breakage, I'd be happy to follow it."}}
{"actor":{"display_login":"thw0rted"}}