-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Trace why the built zip size keeps increasing #5977
Comments
So the size increase has a large component in
|
ughh, looking at
and then noticing that
A single Jest v24 install is 58MB
|
Opened peerigon/parse-domain#91 |
TypeScript is 35~44MB. When we remove jest and TypeScript, we have about 100MB down. Maybe we need bundlesize bot. |
Where are we including jest or typescript as prod dependencies K?
…Sent from my iPhone
On Dec 16, 2019, at 18:27, Kukhyeon Heo ***@***.***> wrote:
TypeScript is 35~44MB. When we remove jest and TypeScript, we have about 100MB down.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
TypeScript is included in And And |
Nice - we found jest in network but we do need fix the other two of course
…Sent from my iPhone
On Dec 16, 2019, at 18:44, Kukhyeon Heo ***@***.***> wrote:
TypeScript is included in server with dependency-tree from #1463 (to watch plugin.js dependencies). dependency-tree uses filing-cabinet and precinct and they use typescript.
And network uses parse-domain and it requires jest and jest family is included in network node_modules.
And @babel family is included to server by @cypress/browserify-preprocessor.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
And when we're building Cypress, we install dependencies to each package. Because of that, we're including some packages several times like We can remove these redundant dependencies by merging the list of dependencies into one list under |
That’s something I was thinking about too - makes little sense to have them separate - and if we move all prod dependencies to packages root it would solve nested package build problem with electron builder
…Sent from my iPhone
On Dec 16, 2019, at 18:56, Kukhyeon Heo ***@***.***> wrote:
And when we're building Cypress, we install dependencies to each package. Because of that, we're including some packages several times like bluebird in 8 packages (server, proxy, network, launcher, https-proxy, extension, example, electron), lodash in 7 packages (electron, extension, https-proxy, launcher, network, proxy, server).
We can remove these redundant dependencies by merging the list of dependencies into one list under packages/package.json and install them from it. Then, we can reduce them.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
Isn't the yarn PR meant to address some of the redundant dependencies? cc @andrew-codes #5555 |
Ughh,
|
|
I created TypeScript issue in #6051. |
This issue has not had any activity in 180 days. Cypress evolves quickly and the reported behavior should be tested on the latest version of Cypress to verify the behavior is still occurring. It will be closed in 14 days if no updates are provided. |
This issue has been closed due to inactivity. |
The current zip of the Test Runner build in the Firefox branch is 191MB! Need to investigate what is driving the increases in size
Example for Mac
Find cached Cypress binary path and versions
You can see the size history by version
Take any version and see the resources sizes - for example 3.2.0 vs 3.8.0. Tip:
sort -h
sorts by human size in the first columnand for 3.8.0
The text was updated successfully, but these errors were encountered: