react-lottie icon indicating copy to clipboard operation
react-lottie copied to clipboard

Remove outdated NPM engine

Open mcallegari10 opened this issue 6 years ago • 18 comments

According to issue #87 and my team's comments, there's a problem with the required engine of this package. Removing the engine request should be enough for now but if you want to make the requirement of any version in specific, should be at least node version 8.

mcallegari10 avatar Sep 16 '19 15:09 mcallegari10

Thanks! I'm waiting for it!

sciotta avatar Sep 26 '19 22:09 sciotta

Would love to see this merged!

dwschrashun avatar Feb 19 '20 17:02 dwschrashun

@dwschrashun I have forked this with some updates :) This is included in the latest version.

https://www.npmjs.com/package/react-lottie-wrapper

donovanclarke avatar Mar 08 '20 17:03 donovanclarke

need to be merged ;)

bsvobodny avatar Mar 31 '20 16:03 bsvobodny

Still not merged. Facing same issue while installing with latest npm.

Unsupported engine for [email protected]: wanted: {"npm":"^3.0.0"} (current: {"node":"14.16.1","npm":"6.14.12"})

AlauddinTheWonder avatar Apr 26 '21 06:04 AlauddinTheWonder

@AlauddinTheWonder, you are more then welcome to pull my version as it should work :).

I think that it is safe to say this repository has been abandoned.

donovanclarke avatar Apr 26 '21 16:04 donovanclarke

Any chance this can be merged? It's literally a 1 liner...

aminland avatar Sep 18 '21 01:09 aminland

@julia-otran when do you guys planning on merging this?

wsameer avatar Sep 25 '21 13:09 wsameer

Facing this issue as well, would be great to merge this fix!

ralph-dev avatar Oct 03 '21 16:10 ralph-dev

Same as above, facing this issue, please merge these changes!

lucferbux avatar Nov 05 '21 19:11 lucferbux

Also facing this issue, please merge!

harry-isaac avatar Nov 06 '21 00:11 harry-isaac

any news about the merge?

mmelilli avatar Mar 24 '22 14:03 mmelilli

Hi. Would love this to be merged @chenqingspring. Thank you

elpatronaco avatar Mar 31 '22 07:03 elpatronaco

Dear maintainers. If you don't intend to continue supporting this repo. Can you pass the ownership to someone else? We need this to move forward with our lives. Thanks!

GCSBOSS avatar Jun 20 '22 14:06 GCSBOSS

Hi, @chenqingspring , @mcallegari10 , it would be great if you could merge this.

sabaimran avatar Sep 27 '22 06:09 sabaimran

Hey, @chenqingspring , @mcallegari10 , Can we please merge this? Thanks in advance!

snaruto7 avatar Oct 10 '22 21:10 snaruto7

Is there any solutions out there? I am having trouble deploying an app without -f options in npm install.

uw-hermlunaiv avatar Oct 16 '22 15:10 uw-hermlunaiv