-
Notifications
You must be signed in to change notification settings - Fork 4
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
Introduce Javy.JSON builtins #16
Conversation
This commit introduces `Javy.JSON` builtins and bumps the package version to 1.0.0. It's important to note that this is a breaking change because it introduces new JavaScript builtins for faster JSON parsing and stringifying from stdin and to stdout. In order to use v1.0.0 of this package, users need the latest shopify cli, which contains `javy` v3.1.0
"peerDependencies": { | ||
"javy": "^0.1.0" |
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.
Why did we drop the peer dependency? 👀
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.
The javy
npm dependency is what brings the fs
namespace, which is no longer used (fs.{readSync, writeSync}
).
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.
Currently unable to build extensions on the linked CLI branch- will circle back to this once that's resolved: Shopify/cli#4444 (comment)
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.
Tophatted 👍
This commit ensures that new functions are bootstrapped with version 1.0.0 of the `@shopify/function_javascript` package, which includes the changes in Shopify/shopify-function-javascript#16 To 🎩 * Create a new app and JavaScript extension. * Build the app * Run the app
This commit introduces
Javy.JSON
builtins and bumps the package version to 1.0.0.It's important to note that this is a breaking change because it introduces new JavaScript builtins for faster JSON parsing and stringifying from stdin and to stdout.
In order to use v1.0.0 of this package, users need the latest shopify cli, which contains
javy
v3.1.0🎩
@shopify/shopify_functionv0.1.0
in the package.json, with a GitHub dependency ("@shopify/shopify_function": "saulecabrera/shopify-function-javascript#javy-json-v1"
) or a local dependency (by cloning my repo and switching to this branch)