Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Sep 26, 2018

Fixes #27373

Separates RenameInfoSuccess and RenameInfoFailure. Previously RenameInfo contained their combined properties -- marked non-optional but we were passing undefined! to them (was a TODO for #18217).
This will technically be a type-level breaking change, but shouldn't be a breaking change at runtime.

@ghost ghost force-pushed the renameInfoUnion branch from 3efc14b to 17570a6 Compare September 26, 2018 23:59
@ghost ghost force-pushed the renameInfoUnion branch from 17570a6 to 9f967c8 Compare September 27, 2018 00:05
@ghost ghost merged commit 5f2741b into master Sep 27, 2018
@ghost ghost deleted the renameInfoUnion branch September 27, 2018 15:56
ghost pushed a commit that referenced this pull request Sep 27, 2018
@ghost ghost mentioned this pull request Sep 27, 2018
ghost pushed a commit that referenced this pull request Oct 11, 2018
mrcrane pushed a commit to mrcrane/TypeScript that referenced this pull request Jul 26, 2019
@microsoft microsoft locked as resolved and limited conversation to collaborators Oct 21, 2025
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant