Skip to content
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

Dependency updates. #532

Merged
merged 1 commit into from Feb 21, 2019
Merged

Dependency updates. #532

merged 1 commit into from Feb 21, 2019

Conversation

coreyfarrell
Copy link
Contributor

  • Replace core-js with the much smaller set-immediate-shim.
  • Remove es6-promise as it is unused.
  • Update lie and readable-stream.

The change from core-js to set-immediate-shim reduces the install size
of jszip from 9.1MB to 2.2MB. The ponyfil for setImmediate is all that
was used from core-js.

* Replace core-js with the much smaller set-immediate-shim.
* Remove es6-promise as it is unused.
* Update lie and readable-stream.

The change from core-js to set-immediate-shim reduces the install size
of jszip from 9.1MB to 2.2MB.  The ponyfil for setImmediate is all that
was used from core-js.
@coreyfarrell
Copy link
Contributor Author

@chuyik the tests passed here (CI is enabled). Unfortunately for me a scoped fork cannot help me as jszip is an indirect dependency via selenium-webdriver.

@Stuk Stuk merged commit 56574c3 into Stuk:master Feb 21, 2019
@Stuk
Copy link
Owner

Stuk commented Feb 21, 2019

I'll publish this as version 3.2.0 in the morning PST.

Unfortunately there's been some bitrot and the Saucelabs configurations are out of date, meaning I can't confirm that this works across all browsers. Looking at the change I think it should be ok.

@coreyfarrell
Copy link
Contributor Author

@Stuk thanks, maybe #506 could be included as well?

@coreyfarrell coreyfarrell deleted the updates branch February 21, 2019 11:42
@Stuk
Copy link
Owner

Stuk commented Feb 21, 2019

Done and done!

@jimmywarting
Copy link
Contributor

Grate to see @Stuk working on jszip again 👍😃
welcome back

@PrzemyslawKlys
Copy link

image

Those files, downloaded from newest release 3.2.0 seem to contain

image

Are those updated or a mistake?

@Stuk
Copy link
Owner

Stuk commented Mar 22, 2019

Thanks for catching. I've published 3.2.1 with the correct dist files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants