Skip to content

Conversation

@yusinto
Copy link
Contributor

@yusinto yusinto commented Apr 17, 2023

To enable sharing of common code between cloudflare and vercel, this PR:

  • moves common code for the cloudflare and vercel sdks to a new package under packages/sdk-server-edge.
  • the cloudflare project is refactored to use this package.
  • sendEvents is turned on and hanging jest tests have been fixed.

Remaining tasks which will be covered in separate PRs:

  • add inline comments to fix typedoc
  • add example cloudflare app

This PR should unblock the vercel team with their work. The vercel sdk should follow the cloudflare sdk requiring only the implementation featureStore and platformInfo objects.

@shortcut-integration
Copy link

This pull request has been linked to Shortcut Story #197407: Refactor common edge function code.

"/dist"
],
"scripts": {
"doc": "../../../scripts/build-doc.sh .",
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 is useful for local dev.

@@ -0,0 +1,3 @@
# Changelog
Copy link
Member

Choose a reason for hiding this comment

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

I probably need to write a document, but when we make the first release you will have to manually edit this file, because release-please will put the release note before this notice. But once you move it to the correct place all future releases will be correct. This is just a note, note a request to change anything.

@yusinto yusinto requested a review from kinyoklion April 17, 2023 23:28
@yusinto yusinto mentioned this pull request Apr 18, 2023
@yusinto yusinto merged commit 0578190 into main Apr 18, 2023
@yusinto yusinto deleted the yus/sc-197407/refactor-common-edge-function-code branch April 18, 2023 16:54
@github-actions github-actions bot mentioned this pull request Apr 18, 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