Skip to content
This repository has been archived by the owner on Jul 24, 2019. It is now read-only.

Force a fixed version for request dependency due to non-availabilit… #783

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nick13jaremek
Copy link

…y of async/await function sin NodeJS versions 6.x/7x.

request upgraded its version from 2.83.0 to 2.84.0, commiting a version change on its package.json file for the hawk NPM dependency, from 6.x to 7.x.

Such hawk dependency version refers to the sntp library, which uses async/await statements in its codebase.

Due to this dependency chain, new installs of phantomjs fail in environments that use NodeJS versions 6.x or 7.x, since these node versions do not support the async/await functions by default.

…y of async/await function sin NodeJS versions 6.x/7x.

`request` upgraded its version from 2.83.0 to 2.84.0, commiting a version change on its *package.json* file for the `hawk` NPM dependency, from 6.x to 7.x.

Such `hawk` dependency version refers to the `sntp` library, which uses *async/await* statements in its codebase.

Due to this dependency chain, new installs of `phantomjs` fail in environments that use NodeJS versions 6.x or 7.x, since these node versions do not support the async/await functions by default.
@nick13jaremek
Copy link
Author

Will require version upgrade and NPM publishing of the newest version.

@Wlada
Copy link

Wlada commented Mar 12, 2018

Merge?
Thanks

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.

2 participants