search

hughsk / vinyl-map

Map vinyl files' contents as strings
MIT License
36 stars 7 forks source link

Updates package.json resolves #12 & #13 #14

Open andrewodri opened 8 years ago

andrewodri commented 8 years ago

Updated package.json dependencies and ran appropriate tests to validate.

Compared bl v0.7.0 to v1.1.2 to ensure API compatibility, and it appears that the module has been optimized considerably since v0.7.0 whilst still remaining compatible.

Removed the branch and ran npm install followed by npm run test; all tests are passing as expected.

dciancu commented 8 years ago

@andrewodri +1 for crafting the PR

@hughsk Hello, thank you very much for your work. Can you please take a look at this PR and merge it, the changes are adequate and it solves npm warnings and errors.

Everyone that relies on this package is currently getting warnings on install and errors on other commands. For example the npm shrinkwrap feature errors because this package requires an old version of the bl package which requires as peer dependency stream-browserify@* - considered invalid by npm.

Cheers.