Skip to content
This repository was archived by the owner on Jul 1, 2026. It is now read-only.

Add NodeJS 11 - #1072

Merged
kewde merged 2 commits into
TryGhost:masterfrom
kewde:nodejs11
Nov 9, 2018
Merged

Add NodeJS 11#1072
kewde merged 2 commits into
TryGhost:masterfrom
kewde:nodejs11

Conversation

@kewde

@kewde kewde commented Oct 30, 2018

Copy link
Copy Markdown
Collaborator

No description provided.

@kewde kewde mentioned this pull request Oct 30, 2018
@kibertoad

Copy link
Copy Markdown

Maybe appveyor CI could be extracted into a separate PR to unblock the fix?

@kibertoad

Copy link
Copy Markdown

@kewde Could you rerun tests? I think appveyor got Node 11 already.

@panther7

panther7 commented Nov 5, 2018

Copy link
Copy Markdown

Any news?

@panther7

panther7 commented Nov 7, 2018

Copy link
Copy Markdown

I hate this ignoring, maintener?

appveyor doesn't have NodeJS 11 support yet
@kewde

kewde commented Nov 9, 2018

Copy link
Copy Markdown
Collaborator Author

@kibertoad rerunning.

@kibertoad

Copy link
Copy Markdown

@kewde Node 11 passed, some other random failures happened. Probably needs to be triggered again (btw, you don't need comment for that, closing and reopening the PR does the trick)

@kewde
kewde merged commit 876b99c into TryGhost:master Nov 9, 2018
@kewde
kewde deleted the nodejs11 branch November 9, 2018 21:01
@kewde

kewde commented Nov 9, 2018

Copy link
Copy Markdown
Collaborator Author

@kibertoad I've merged the NodeJS 11 code and started the binary publishing process.
It will take a few hours before these are available.

@kewde

kewde commented Nov 10, 2018

Copy link
Copy Markdown
Collaborator Author

Still waiting for version 4.0.4 to be published to npm though. See #1077

@panther7

Copy link
Copy Markdown

Thanks

@springmeyer

springmeyer commented Nov 10, 2018

Copy link
Copy Markdown
Contributor

Still waiting for version 4.0.4 to be published to npm though. See #1077

This is now done

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants