Start running tests on Node.js v12. #2983
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
While not currently a so-called "Long-Term-Support" (LTS) release, Node.js v12 is on target to become LTS in October 2019 and Node.js 8 will end LTS earlier than previous even-numbered Node.js releases. See the Node.js schedule: https://github.com/nodejs/Release#release-schedule With that only a few months away, it's time to start getting a read on whether we're going to pass tests in that new engine.
…ely. While originally introduced in #92 to accommodate Meteor, it seems that we're no longer making `Promise`s Fiber-aware using the `meteor-promise` library which leverages `fibers` under-the-hood. Since this package seems to be the sole reason that bootstrapping is failing on Node.js 12. To support Node.js 12, we could update to `fibers@4` which accommodates changes to the V8 API which have been made in recent Node.js versions, the `fibers@4` implementation is only compatible with Node.js 10 and 12 and is incompatible with Node.js 8, which we still actively support (though not for much longer!).
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
While not currently a so-called "Long-Term-Support" (LTS) release, Node.js v12 is on target to become LTS in October 2019 and Node.js 8 will end LTS earlier than previous even-numbered Node.js releases.
See the Node.js schedule: https://github.com/nodejs/Release#release-schedule
With that only a few months away, it's time to start getting a read on whether we're going to pass tests in that new engine.
This required the removal of
meteor-promiseandfibersfrom the rootpackage.jsonmanifest - sincefibers@3doesn't support Node.js 12 butfibers@4doesn't support Node.js 8 — a temporary conundrum until we drop Node.js 8 support in Apollo Server 3!), but those packages were entirely unused anyhow, so we're not losing anything by doing this, though I'm certainly questioning when that removal happened and how it might have affected Meteor users!