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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Miniflare and workerd dependencies #1866
Conversation
@@ -890,7 +890,7 @@ | |||
"isOptional": true | |||
} | |||
], | |||
"value": "interface ExternalVideoBaseProps {\n /**\n * An object with fields that correspond to the Storefront API's [ExternalVideo object](https://shopify.dev/api/storefront/reference/products/externalvideo).\n */\n data: PartialDeep<ExternalVideoType, {recurseIntoArrays: true}>;\n /** An object containing the options available for either\n * [YouTube](https://developers.google.com/youtube/player_parameters#Parameters) or\n * [Vimeo](https://vimeo.zendesk.com/hc/en-us/articles/360001494447-Using-Player-Parameters).\n */\n options?: YouTube | Vimeo;\n}" | |||
"value": "export interface ExternalVideoBaseProps {\n /**\n * An object with fields that correspond to the Storefront API's [ExternalVideo object](https://shopify.dev/api/storefront/reference/products/externalvideo).\n */\n data: PartialDeep<ExternalVideoType, {recurseIntoArrays: true}>;\n /** An object containing the options available for either\n * [YouTube](https://developers.google.com/youtube/player_parameters#Parameters) or\n * [Vimeo](https://vimeo.zendesk.com/hc/en-us/articles/360001494447-Using-Player-Parameters).\n */\n options?: YouTube | Vimeo;\n}" |
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.
Is anyone familiar with the generated docs here? These changes add export
to some of the internal interfaces/types and I'm not 100% this is fine with the docs 馃
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's fine. It looks like part of type doc generation.
Closes #1833
There's a change in Node 21.7.0 that makes Miniflare throw. This has been fixed upstream so we just need to update the dependency.
馃帺 :
main
,nvm install 21.7
npm run dev
in skeleton template, you'll see:While I was updating this dep, I fixed other inconsistencies with our dependencies (multipass example was using old Oxygen worker types) and deduped the package-lock. As a result, I think we are now using a newer version of TS for development and a few type issues came up. I've fixed those as well by refactoring a bit hydrogen-react and hydrogen.