netlify / blobs

A TypeScript client for Netlify Blobs
https://docs.netlify.com/blobs/overview
MIT License
16 stars 5 forks source link

feat: remove usage of Node built-ins #154

Closed eduardoboucas closed 8 months ago

eduardoboucas commented 8 months ago

Which problem is this pull request solving?

Replaces any Node built-in modules with cross-runtime equivalents.

The exception is the local server, which is Node only.

netlify[bot] commented 8 months ago

Deploy Preview for blobs-js ready!

Name Link
Latest commit 741225f2c3b8b9a4ae0553bd026f5f2650b0f06c
Latest deploy log https://app.netlify.com/sites/blobs-js/deploys/65e89301435a2a00088e6386
Deploy Preview https://deploy-preview-154--blobs-js.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

ascorbic commented 8 months ago

I wonder if the server could be moved to a subpath?

eduardoboucas commented 8 months ago

I wonder if the server could be moved to a subpath?

I think so, but I'd rather do that as a follow-up because I want to flag it as a breaking change.