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

chore: upgrade vm-browserify to 1.0.1 #77

Merged
merged 3 commits into from
Jun 12, 2019

Conversation

fahrradflucht
Copy link
Contributor

@fahrradflucht fahrradflucht commented Apr 27, 2018

vm-browserify up until 1.0.1 depends upon unlicensed code. [0][1] This means
node-libs-browser does too. This resolves the issue by upgrading it. All
changes in between were non-breaking. [2]

[0] browserify/browserify#1828
[1] component/indexof#6
[2] https://github.com/browserify/vm-browserify/releases

vm-browserify up until 1.0.1 dependy upon unlicensed code. [0][1] This means
node-libs-browser does too. This resolves the issue by upgrading it. All
changes in between where non-breaking. [2]

[0] browserify/browserify#1828
[1] component/indexof#6
[2] https://github.com/browserify/vm-browserify/releases
@jsf-clabot
Copy link

jsf-clabot commented Apr 27, 2018

CLA assistant check
All committers have signed the CLA.

@opichals
Copy link

opichals commented Aug 1, 2018

@fahrradflucht Please update to 1.1.0 as there is no indexof dependency anymore inside.

@jhnns @sokra Could you have a look at this? It is unlicensed code issue affecting the whole webpack ecosystem.

@fahrradflucht
Copy link
Contributor Author

fahrradflucht commented Aug 1, 2018

@opichals There is no difference in dependencies between 1.0.1 and 1.1.0. The licensing issue is fixed by this MR. If new features are wanted (vm.isContext(sandbox)) then I could update the MR but my intention was to fix the licensing problem.

@johnwebbcole
Copy link

Any chance of getting this merged soon?

Thx,

J

package.json Outdated Show resolved Hide resolved
Copy link
Contributor

@goto-bus-stop goto-bus-stop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this update is safe and good to go 👍

@sokra sokra merged commit c88c6b0 into webpack:master Jun 12, 2019
@sokra
Copy link
Member

sokra commented Jun 12, 2019

Thanks

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.

6 participants