deprecated dependencies with NPM Installation

phpcodingmaster commented 1 month ago

CheckList

[X] I agree to follow this project's Code of Conduct
[X] I have read and followed the Contributing Guide
[X] I have read and followed the Issue Tracker Guide
[X] I have searched and referenced existing issues and discussions
[X] I am filing a BUG report.
[X] I have managed to reproduce the bug after upgrading to the latest version
[X] I have created an accurate and minimal reproduction

Version

6.0.2

In What environments are you experiencing the problem?

Node.js

Node Version (if applicable)

None

Link To Reproduction

https://codesandbox.io/p/devbox/fabric-node-sandbox-forked-xewlhg

Steps To Reproduce

Run npm install fabric or npm install fabric --save
Check the console

Expected Behavior

Should not be deprecated modules.

Actual Behavior

npm warn deprecated inflight@1.0.6: This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful. npm warn deprecated npmlog@5.0.1: This package is no longer supported. npm warn deprecated rimraf@3.0.2: Rimraf versions prior to v4 are no longer supported npm warn deprecated glob@7.2.3: Glob versions prior to v9 are no longer supported npm warn deprecated are-we-there-yet@2.0.0: This package is no longer supported.

Error Message & Stack Trace

No response

asturur commented 1 month ago

Those modules are part of the dev tooling for fabric. Did you try to use npm install --production to avoid install dev deps?

phpcodingmaster commented 1 month ago

Hi @asturur - If I try --production it still shows the below.

npm warn deprecated npmlog@5.0.1: This package is no longer supported. npm warn deprecated rimraf@3.0.2: Rimraf versions prior to v4 are no longer supported npm warn deprecated glob@7.2.3: Glob versions prior to v9 are no longer supported npm warn deprecated abab@2.0.6: Use your platform's native atob() and btoa() methods instead npm warn deprecated are-we-there-yet@2.0.0: This package is no longer supported. npm warn deprecated domexception@4.0.0: Use your platform's native DOMException instead npm warn deprecated gauge@3.0.2: This package is no longer supported.

Seems like there are a lot of packages that are deprecated

asturur commented 1 month ago

If they are deprecated are deprecated. I m not investing time in the dev deps unless is JEST or is JSDOM. If you want to discover which one are deprecated and from what and what it takes to update them you can open a PR.

gloriousjob commented 1 month ago

These don't appear to be direct dependencies of fabric.js but indirect. Like are-we-there-yet appears to be a dependency of npmlog,which appears to be a dependency of testem. So this would really be a an issue of those libraries.

phpcodingmaster commented 3 weeks ago

Ok.

@gloriousjob - thank you! @asturur - thank you for the reply.

fabricjs / fabric.js