Skip to content

Conversation

@yusinto
Copy link
Contributor

@yusinto yusinto commented Apr 19, 2023

  • add example test app for cloudflare
  • adds support for cjs and esm bundles for sdk-server-common and cloudflare

@shortcut-integration
Copy link

This pull request has been linked to Shortcut Story #196410: Bundle cloudflare output.

@yusinto yusinto mentioned this pull request Apr 19, 2023
@yusinto yusinto requested a review from InTheCloudDan April 19, 2023 17:37
"lib": [
"es2021"
] /* Specify a set of bundled library declaration files that describe the target runtime environment. */,
"jsx": "react" /* Specify what JSX code is generated. */,
Copy link
Member

Choose a reason for hiding this comment

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

Do we need jsx for some reason?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This comes default from wrangler init. We don't need jsx in our example app, but maybe others may in their apps.

Copy link
Member

@kinyoklion kinyoklion left a comment

Choose a reason for hiding this comment

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

Approved with comments.

@yusinto yusinto merged commit bab593c into main Apr 19, 2023
@yusinto yusinto deleted the yus/sc-196410/bundle-cloudflare-output branch April 19, 2023 21:27
@github-actions github-actions bot mentioned this pull request Apr 19, 2023
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.

3 participants