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 doc comment for the Except type #191

Merged
merged 2 commits into from
Mar 20, 2021
Merged

Improve the doc comment for the Except type #191

merged 2 commits into from
Mar 20, 2021

Conversation

shiftgeist
Copy link
Contributor

After much discussion, we think libraries providing their own "stricter" versions of Omit (as well as Exclude and other friends) is preferable.

microsoft/TypeScript#30825 (comment)

Since the question is decided and the issue closed, the comment may be removed.

@sindresorhus
Copy link
Owner

I think it should still link to the TS issue, but instead the text should be the resolution; the fact that the TS team has declined a stricter version and why.

@shiftgeist
Copy link
Contributor Author

I changed the comment.

@sindresorhus sindresorhus changed the title Remove comment in except.d.ts Improve the doc comment for the Except type Mar 20, 2021
@sindresorhus sindresorhus merged commit 8bd5d27 into sindresorhus:main Mar 20, 2021
@shiftgeist shiftgeist deleted the patch-except branch March 20, 2021 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants