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

[CS2] Require Node 6.9.1+ #4341

Merged
merged 3 commits into from
Oct 21, 2016
Merged

Conversation

GeoffreyBooth
Copy link
Collaborator

@GeoffreyBooth GeoffreyBooth commented Oct 19, 2016

Node 6 deprecated new Buffer in favor of Buffer.from and Buffer.alloc. This PR updates our calls to Buffer to use the new syntax. As a result of that, I also set the required version of Node per package.json to be >= 6.9.1, and set the overall version of the project to be 2.0.0-alpha for now.

…alloc`; update our calls, and set the required version of Node to be >= 6.9.0, and set this to be 2.0.0-alpha for now
@GeoffreyBooth GeoffreyBooth added this to the 2.0.0 milestone Oct 19, 2016
@GeoffreyBooth GeoffreyBooth changed the title Node 6 deprecated new Buffer in favor of Buffer.from and `Buffer.… new Buffer is deprecated in Node 6 Oct 19, 2016
@lydell
Copy link
Collaborator

lydell commented Oct 19, 2016

LGTM. Does it make sense to do something in master as well? For instance, (Buffer.from?(src) ? new Buffer(src)).toString('base64').

@GeoffreyBooth
Copy link
Collaborator Author

Hmm, that’s clever. I suppose we could, though I’m not sure what benefit it would provide. new Buffer doesn’t throw a deprecation warning (yet), even in Node nightly.

@GeoffreyBooth GeoffreyBooth changed the title new Buffer is deprecated in Node 6 [CS2] Require Node 6.9.0+ Oct 19, 2016
@GeoffreyBooth
Copy link
Collaborator Author

Should I maybe use a package.json version number like 2.0.0-nightly? or 2.0.0-alpha0? To imply that this branch will keep changing frequently without version bumps. Or keep 1.11.1?

@jashkenas
Copy link
Owner

2.0.0-alpha sounds about right to me.

@GeoffreyBooth GeoffreyBooth changed the title [CS2] Require Node 6.9.0+ [CS2] Require Node 6.9.1+ Oct 21, 2016
@GeoffreyBooth
Copy link
Collaborator Author

Any objections to merging this in?

@lydell
Copy link
Collaborator

lydell commented Oct 21, 2016

Not from me.

@GeoffreyBooth GeoffreyBooth merged commit 8b50fd0 into jashkenas:2 Oct 21, 2016
@GeoffreyBooth GeoffreyBooth deleted the buffer-deprecations branch October 21, 2016 16:56
gluons added a commit to gluons/gulp-json2cson that referenced this pull request Jan 13, 2018
- Because CoffeeScript 2 requires Node 6+. (Ref:
jashkenas/coffeescript#4341)
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.

3 participants