Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

hapi warnings when installing 0.35.0 #2008

Closed
aphelionz opened this issue Apr 25, 2019 · 2 comments · Fixed by #2015
Closed

hapi warnings when installing 0.35.0 #2008

aphelionz opened this issue Apr 25, 2019 · 2 comments · Fixed by #2015
Assignees
Labels
exp/novice Someone with a little familiarity can pick up P3 Low: Not priority right now

Comments

@aphelionz
Copy link
Contributor

  • Version: 0.35.0
  • Platform: node.js
  • Subsystem: dependencies

Type: Bug

Severity: Low

Description:

When installing js-ipfs@^0.35.0 I see the following trace of errors from hapi and its submodules.

image

Steps to reproduce the error:

Simply running npm i ipfs should trigger these warnings

@alanshaw
Copy link
Member

alanshaw commented May 3, 2019

Thanks for reporting, this should hopefully be easy to fix!

@alanshaw alanshaw added exp/novice Someone with a little familiarity can pick up P3 Low: Not priority right now status/ready Ready to be worked labels May 3, 2019
alanshaw pushed a commit that referenced this issue May 3, 2019
resolves #2008

License: MIT
Signed-off-by: Alan Shaw <[email protected]>
@ghost ghost assigned alanshaw May 3, 2019
@ghost ghost added status/in-progress In progress and removed status/ready Ready to be worked labels May 3, 2019
@alanshaw
Copy link
Member

alanshaw commented May 4, 2019

Fixed for this module in #2015 but a couple of dependencies are still using the old version and need to be upgraded. I've opened issues:

@ghost ghost removed the status/in-progress In progress label May 4, 2019
alanshaw pushed a commit that referenced this issue May 4, 2019
Resolves #2008 for this module but a couple of dependencies are still using the old version and need to be upgraded. I've opened issues:

* libp2p/js-libp2p-webrtc-star#172
* ipfs/js-ipfsd-ctl#331

License: MIT
Signed-off-by: Alan Shaw <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
exp/novice Someone with a little familiarity can pick up P3 Low: Not priority right now
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants