Skip to content

Optional updated#165

Open
baharalidurrani wants to merge 4 commits into
piotrwitek:masterfrom
baharalidurrani:optional-updated
Open

Optional updated#165
baharalidurrani wants to merge 4 commits into
piotrwitek:masterfrom
baharalidurrani:optional-updated

Conversation

@baharalidurrani

Copy link
Copy Markdown

Description

Updated Optional Utility type so that it can strictly check the names of the keys that we want to make theme Optional in the new resultant type.

Related issues:

  • None

Checklist

  • I have read CONTRIBUTING.md
  • I have linked all related issues above
  • I have rebased my branch

For bugfixes:

  • I have added at least one unit test to confirm the bug have been fixed
  • I have checked and updated TOC and API Docs when necessary

For new features:

  • I have added entry in TOC and API Docs
  • I have added a short example in API Docs to demonstrate new usage
  • I have added type unit tests with dts-jest

@baharalidurrani baharalidurrani left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

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.

1 participant