Skip to content

chore(travis) add node 12 - #3946

Merged
dougwilson merged 1 commit into
expressjs:4.17from
zuohuadong:patch-1
May 16, 2019
Merged

chore(travis) add node 12#3946
dougwilson merged 1 commit into
expressjs:4.17from
zuohuadong:patch-1

Conversation

@zuohuadong

Copy link
Copy Markdown

No description provided.

@dougwilsondougwilson left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please update the appveyor.yml as well 👍

@dougwilsondougwilson self-assigned this May 9, 2019
@dougwilson

Copy link
Copy Markdown
Contributor

@zuohuadong ?

@zuohuadong

Copy link
Copy Markdown
Author

@zuohuadong ?

Sorry, I didn't see it before.

@zuohuadong

Copy link
Copy Markdown
Author

@zuohuadong ?
I've already finished.

@zuohuadong

Copy link
Copy Markdown
Author

Should we remove some versions that are no longer officially supported by node.js?

@LinusU

Copy link
Copy Markdown
Member

Should we remove some versions that are no longer officially supported by node.js?

We can only remove support for older Node.js runtimes in a breaking change, so that will have to happen on the 5.x branch...

@dougwilsondougwilson mentioned this pull request May 16, 2019
23 tasks
@dougwilsondougwilson added this to the 4.17 milestone May 16, 2019
@dougwilsondougwilson changed the title chore(travis) add node 10 11 12chore(travis) add node 12May 16, 2019
@dougwilson
dougwilson changed the base branch from master to 4.17May 16, 2019 04:59
@dougwilson

Copy link
Copy Markdown
Contributor

I re-targeted the PR to the 4.17 branch and only 12 was not covered, so just updated the PR to be adding Node.js 12.

@dougwilson
dougwilson merged commit b9ecb9a into expressjs:4.17May 16, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@zuohuadong@dougwilson@LinusU