search

cloudinary / cloudinary_npm

Cloudinary NPM for node.js integration
629 stars 323 forks source link

Q dependency is deprecated #686

Open dmythro opened 2 months ago

dmythro commented 2 months ago

Bug report for Cloudinary NPM SDK

Before proceeding, please update to latest version and test if the issue persists

Describe the bug in a sentence or two.

I get this warning on every build:

npm warn deprecated q@1.5.1: You or someone you depend on is using Q, the JavaScript Promise library that gave JavaScript developers strong feelings about promises. They can almost certainly migrate to the native JavaScript promise now. Thank you literally everyone for joining me in this bet against the odds. Be excellent to each other.
npm warn deprecated
npm warn deprecated (For a CapTP with native promises, see @endo/eventual-send and @endo/captp)

Is there a reason why native promises aren't used yet?

Issue Type (Can be multiple)

[ ] Build - Can’t install or import the SDK [ ] Babel - Babel errors or cross browser issues [x] Performance - Performance issues [ ] Behaviour - Functions aren’t working as expected (Such as generate URL) [ ] Documentation - Inconsistency between the docs and behaviour [ ] Incorrect Types - For typescript users who are having problems with our d.ts files [x] Other (Specify)

Versions and Libraries (fill in the version numbers)

Cloudinary_NPM SDK version 2.4.0 Node - 20.17.0 NPM - 10.8.2

wissam-khalili commented 2 months ago

Hi @dmythro ,

Thank you for your input. We will review the issue and will keep you posted on our findings.

Best Regards, Wissam

Yashwanth-Chandrakumar commented 1 month ago

Hi @wissam-khalili can you assign it to me i will do it. Because it has been stale for long