Skip to content

Update package.json engines to reflect lack of 10 support#1543

Closed
maxkorp wants to merge 1 commit into
masterfrom
node-v10-message
Closed

Update package.json engines to reflect lack of 10 support#1543
maxkorp wants to merge 1 commit into
masterfrom
node-v10-message

Conversation

@maxkorp

@maxkorp maxkorp commented Aug 29, 2018

Copy link
Copy Markdown
Collaborator

Doesnt mean much directly, except npm will warn and yarn I think will error if you try to install with node 10. Just to provide a bit of feedback to people running into that, since it seems to be coming up a lot.

@tbranyen

Copy link
Copy Markdown
Member

I think we're close to getting Node 10 support, so I may want to hold off on this. If I'm wrong, we can merge and push out a patch?

@implausible

Copy link
Copy Markdown
Member

We're pretty close. I'm hoping we can wrap it up this week and Node 10 support will arrive at the same time that Node 10 LTS arrives.

@Croydon Croydon mentioned this pull request Oct 1, 2018
@maxkorp

maxkorp commented Oct 3, 2018

Copy link
Copy Markdown
Collaborator Author

WHAT YOU DON'T WANT TO MERGE THIS NOW THAT IT'S WRONG?!

@tbranyen tbranyen deleted the node-v10-message branch October 3, 2018 21:47
@tbranyen

tbranyen commented Oct 3, 2018

Copy link
Copy Markdown
Member

I even went so far as to delete you branch, let this be a lesson about being wrong. :trollface:

@implausible

Copy link
Copy Markdown
Member

😄

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