-
Notifications
You must be signed in to change notification settings - Fork 52
fix: Typo in type declarations #634
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
Conversation
b620bb7
to
59d2031
Compare
@villelahdenvuo This should fix the type errors you are having. |
Codecov Report
@@ Coverage Diff @@
## main #634 +/- ##
=======================================
Coverage 68.68% 68.68%
=======================================
Files 134 134
Lines 6004 6004
Branches 1144 1144
=======================================
Hits 4124 4124
Misses 1544 1544
Partials 336 336
Flags with carried forward coverage won't be shown. Click here to find out more. 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Asset Size ReportMerging this pull request will result in the following CDN asset size changes:
Merging this pull request will result in the following NPM package consumer size changes:
Other Standard CDN AssetsReleased Assets
Built Assets
Other Polyfill CDN AssetsReleased Assets
Built Assets
|
Nice to see tests added. 😃 |
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.
LGTM
Fixing typos made in the browser and micro agent API declarations. This includes adding internal test cases for those types to ensure this issue does not happen again.
Overview
This fixes typos made in the agent API type declarations and adds tests for those types.
Related Issue(s)
#613 (comment)
Testing
Run `npm run npm:build:types && npm run test:types"