Skip to content
This repository has been archived by the owner on Apr 16, 2021. It is now read-only.

Build web, node, and production bundle #338

Closed
wants to merge 15 commits into from
Closed

Build web, node, and production bundle #338

wants to merge 15 commits into from

Conversation

mrcnski
Copy link
Contributor

@mrcnski mrcnski commented Feb 23, 2021

I've moved to Webpack and created two separate targets for node and dev.

TODO (followup PRs)

Production build deploy

  1. The production web build should be deployed to our skynet-js HNS. I plan to include a deploy.js script in the repo which uses the Node version of the library itself to upload the production file and save the skylink to the registry. (Deploy production builds when publishing #351)

Update docs

  1. Remove nodejs-skynet SDK from the docs (and archive repo)
  2. Update nodejs examples
  3. Add notes about differences in API between Node and Web (since the former lacks File and the latter lacks access to the local filesystem)

- Output bundles should be called index.node.js and index.web.js to match the TS
declaration files
- plugin-transform-runtime was causing runtime errors. I removed it since it
didn't seem to have an effect on the bundle size anyways
@mrcnski mrcnski changed the title [WIP] Webpack Webpack Mar 2, 2021
@mrcnski mrcnski requested a review from ro-tex March 2, 2021 09:52
@mrcnski mrcnski changed the title Webpack Build web, node, and production bundle Mar 2, 2021
@mrcnski mrcnski force-pushed the webpack branch 2 times, most recently from fded92b to 1c456bc Compare March 2, 2021 15:59
@mrcnski
Copy link
Contributor Author

mrcnski commented Mar 31, 2021

Closing in flavor of SkynetLabs/skynet-js#16

@mrcnski mrcnski closed this Mar 31, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant