Skip to content

Commit

Permalink
Remove meteor-promise and fibers, which appear to be unused entir…
Browse files Browse the repository at this point in the history
…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!).
  • Loading branch information
abernix committed Jul 3, 2019
1 parent 11226d8 commit b7ee5ab
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 30 deletions.
27 changes: 0 additions & 27 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 0 additions & 3 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,6 @@
"@types/body-parser": "1.17.0",
"@types/connect": "3.4.32",
"@types/fast-json-stable-stringify": "2.0.0",
"@types/fibers": "0.0.30",
"@types/graphql": "14.2.2",
"@types/hapi": "17.8.6",
"@types/ioredis": "4.0.12",
Expand Down Expand Up @@ -106,7 +105,6 @@
"deep-freeze": "0.0.1",
"express": "4.17.1",
"fastify": "1.14.6",
"fibers": "3.1.1",
"form-data": "2.4.0",
"graphql": "14.4.1",
"graphql-subscriptions": "1.1.0",
Expand All @@ -125,7 +123,6 @@
"lerna": "3.13.4",
"lint-staged": "8.2.1",
"memcached-mock": "0.1.0",
"meteor-promise": "0.8.7",
"mock-req": "0.2.0",
"multer": "1.4.1",
"node-fetch": "2.3.0",
Expand Down

0 comments on commit b7ee5ab

Please sign in to comment.