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

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

Open
SirensOfTitan wants to merge 2 commits intoProjectOpenSea:masterfrom
SirensOfTitan:sirensoftitan/jun-25/reduceSize
Open

Move dependencies around, only distribute npm items in lib/.#3
SirensOfTitan wants to merge 2 commits intoProjectOpenSea:masterfrom
SirensOfTitan:sirensoftitan/jun-25/reduceSize

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