Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updating request module #4

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

Updating request module #4

wants to merge 1 commit into from

Conversation

pdehaan
Copy link

@pdehaan pdehaan commented Oct 9, 2014

There was a known issue with earlier versions of the request module which included [email protected] or less.

There was a known issue with earlier versions of the **request** module which included **[email protected]** or less.
@pdehaan
Copy link
Author

pdehaan commented Oct 9, 2014

Current output:

$ git clone https://github.com/zaach/node-fx-sync.git .

$ npm install
path.existsSync is now called `fs.existsSync`.

$ npm shrinkwrap --dev
wrote npm-shrinkwrap.json

$ # sudo npm i nsp -g
$ nsp audit-shrinkwrap
Name  Installed  Patched  Vulnerable Dependency
qs      0.6.6     >= 1.x  fx-sync > request

$ npm outdated --depth 0
Package         Current  Wanted  Latest  Location
hawk              1.1.2   1.1.2   2.3.0  hawk
hkdf              0.0.1   0.0.1   0.0.2  hkdf
jwcrypto          0.4.4   0.4.4   0.5.2  jwcrypto
p-promise         0.2.5   0.2.5   0.4.8  p-promise
request          2.27.0  2.27.0  2.45.0  request
xmlhttprequest    1.5.1     git     git  xmlhttprequest

# .travis.yml not found

$ # sudo npm i package-json-validator -g
$ pjv -wr
package.json is NOT valid
{ valid: false,
  errors: [ 'Type for field repository, was expected to be object, not string' ],
  warnings:
   [ 'Missing recommended field: bugs',
     'Missing recommended field: contributors' ],
  recommendations:
   [ 'Missing optional field: homepage',
     'Missing optional field: engines' ] }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant