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.

None yet

4 participants