Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve the Omit helper type (#31153) #33523

Closed
wants to merge 5 commits into from
Closed

Improve the Omit helper type (#31153) #33523

wants to merge 5 commits into from

Conversation

vipcxj
Copy link

@vipcxj vipcxj commented Sep 20, 2019

Fixes #31153
Since Omit is the offical helper type now, I think it should be a perfect version. The common version not work with type like { a: string, b: number, [key: string]: any, }, this version works. The detail see #31153

@typescript-bot
Copy link
Collaborator

It looks like you've sent a pull request to update our 'lib' files. These files aren't meant to be edited by hand, as they consist of last-known good states of the compiler and are generated from 'src'. Unless this is necessary, consider closing the pull request and sending a separate PR to update 'src'.

@typescript-bot typescript-bot added the lib update PR modifies files in the `lib` folder label Sep 20, 2019
@vipcxj vipcxj closed this Sep 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lib update PR modifies files in the `lib` folder
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Exclude not work on [key: string]:any
2 participants