-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Packages: Add types directive to api-fetch and date packages #30252
Conversation
Size Change: 0 B Total Size: 1.42 MB ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor thing with documenting but otherwise it looks good 👍🏻
Can you point me to the PR where types were added? I'm wondering if we can backport this PR to the plugin release?
@@ -2,6 +2,8 @@ | |||
|
|||
## Unreleased | |||
|
|||
- Bundle type definitions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It could have a section like:
### New Features
The same applies to the api-fetch
package.
1f67e61
to
ee9b4d5
Compare
* Add types directive to packages that need it * Update date CHANGELOG for types * Fix changelogs
Description
Now that
date
andapi-fetch
are fully typed, we can add thetypes
directive to theirpackage.json
files.How has this been tested?
Build passes
Types of changes
Non breaking changes.
Checklist: