Skip to content
This repository has been archived by the owner on Sep 15, 2022. It is now read-only.

Move dependencies around, only distribute npm items in lib/. #3

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

SirensOfTitan
Copy link

Summary

Moves dependencies around to reduce full size of distribution of package, and only includes compiled files in npm package.

Not including dependency savings (which will be very large for this particular project as wyvern-js depends on typescript in dependencies, where it should be a devDependency only.

#Before: 
npm notice package size:  850.3 kB
npm notice unpacked size: 10.5 MB

#After
npm notice package size:  58.0 kB
npm notice unpacked size: 483.6 kB

Test Plan

  1. yarn link wyvern-js into wyvern-schemas
  2. Ensure wyvern-schemas builds yarn and tests yarn test with all greens.

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