Skip to content

Conversation

@mgol
Copy link
Member

@mgol mgol commented Dec 30, 2016

Summary

Node.js 0.12 is no longer supported by upstream. So far we had to keep
a workaround that would skip running ESLint there (or even load its task)
as it no longer supports Node.js <4; the same applied to Node smoke tests
as jsdom has also dropped support for those old Node versions. Those
workarounds have been removed now.

Ref 030191a

Checklist

Mark an [x] for completed items, if you're not sure leave them unchecked and we can assist.

Thanks! Bots and humans will be around shortly to check it out.

@mgol mgol added the Build label Dec 30, 2016
@mgol mgol requested a review from markelog December 30, 2016 11:17
@mention-bot
Copy link

@mgol, thanks for your PR! By analyzing the history of the files in this pull request, we identified @markelog, @timmywil and @mikesherov to be potential reviewers.

@mgol mgol force-pushed the node0.12-eol branch 2 times, most recently from bbeb533 to 9b3cdd2 Compare December 30, 2016 11:24
@markelog
Copy link
Member

Yey! We shouldn't have any issues with jenkins right?

@mgol
Copy link
Member Author

mgol commented Dec 30, 2016

I switched our project on Jenkins to Node 4 a while ago. :) I should switch it to Node 6 now since Node 4 will get into Maintenance Mode in 4 months.

@markelog
Copy link
Member

Cool

Node.js 0.12 is no longer supported by upstream. So far we had to keep
a workaround that would skip running ESLint there (or even load its task)
as it no longer supports Node.js <4; the same applied to Node smoke tests
as jsdom has also dropped support for those old Node versions. Those
workarounds have been removed now.

Ref 030191a
Closes jquerygh-3478
@mgol mgol merged commit d10ac0e into jquery:master Dec 30, 2016
@mgol mgol deleted the node0.12-eol branch December 30, 2016 12:59
@lock lock bot locked as resolved and limited conversation to collaborators Jan 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Development

Successfully merging this pull request may close these issues.

3 participants