Skip to content
This repository has been archived by the owner on Mar 17, 2021. It is now read-only.

fix(index): use Buffer.from instead of deprecated new Buffer #113

Merged
merged 1 commit into from
Mar 3, 2018

Conversation

ChALkeR
Copy link
Contributor

@ChALkeR ChALkeR commented Feb 22, 2018

Use Buffer.from(string) instead of deprecated new Buffer(string).

Requires Node.js >= 4.5.0, but that seems to fit into current version requirements.

Refs:
https://nodejs.org/api/deprecations.html#deprecations_dep0005_buffer_constructor

Use Buffer.from(string) instead of deprecated new Buffer(string).

Requires Node.js >= 4.5.0, but that seems to fit into current version requirements.

Refs:
https://nodejs.org/api/deprecations.html#deprecations_dep0005_buffer_constructor
@jsf-clabot
Copy link

jsf-clabot commented Feb 22, 2018

CLA assistant check
All committers have signed the CLA.

@michael-ciniawsky michael-ciniawsky changed the title Avoid using deprecated Buffer constructor fix(index): use Buffer.from instead of deprecated new Buffer Feb 22, 2018
Copy link
Member

@michael-ciniawsky michael-ciniawsky left a comment

Choose a reason for hiding this comment

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

@ChALkeR Thx

@michael-ciniawsky
Copy link
Member

Please sign the CLA :)

Copy link
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

👍

@ChALkeR
Copy link
Contributor Author

ChALkeR commented Mar 2, 2018

Tracking: nodejs/node#19079

@alexander-akait
Copy link
Member

/cc @michael-ciniawsky

@michael-ciniawsky michael-ciniawsky merged commit 457618b into webpack-contrib:master Mar 3, 2018
@michael-ciniawsky michael-ciniawsky removed this from the 1.0.0 milestone Mar 3, 2018
@michael-ciniawsky
Copy link
Member

Released in v1.0.0 🎉

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

Successfully merging this pull request may close these issues.

4 participants