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

Linting question #463

Open
liusida opened this issue Apr 3, 2024 · 1 comment
Open

Linting question #463

liusida opened this issue Apr 3, 2024 · 1 comment

Comments

@liusida
Copy link

liusida commented Apr 3, 2024

I am new to the modern Javascript, and here is what I did to enable linting for this project.

  1. git clone this repo. This is simple.
  2. npm install to install all modules. There's no error, but several warnings I'll paste below.
  3. npm run lint:fix to check the source code. I've got ✖ 761 problems (109 errors, 652 warnings)!
    For example, there are many errors on 'clamp' is not defined.

Did I miss any steps? I just want to setup this project in VSCode and read through and learn. Can anyone offer some insights? Thank you very much!

E:\litegraph.js> npm install
npm WARN deprecated cryptiles@0.2.2: This version has been deprecated in accordance with the hapi support policy (hapi.im/support). Please upgrade to the latest version to get the best features, bug fixes, and security patches. If you are unable to upgrade at this time, paid support is available for older versions (hapi.im/commercial).
npm WARN deprecated boom@0.4.2: This version has been deprecated in accordance with the hapi support policy (hapi.im/support). Please upgrade to the latest version to get the best features, bug fixes, and security patches. If you are unable to upgrade at this time, paid support is available for older versions (hapi.im/commercial).
npm WARN deprecated sntp@0.2.4: This module moved to @hapi/sntp. Please make sure to switch over as this distribution is no longer supported and may contain bugs and critical security issues.
npm WARN deprecated request@2.40.0: request has been deprecated, see https://github.com/request/request/issues/3142
npm WARN deprecated hoek@0.9.1: This version has been deprecated in accordance with the hapi support policy (hapi.im/support). Please upgrade to the latest version to get the best features, bug fixes, and security patches. If you are unable to upgrade at this time, paid support is available for older versions (hapi.im/commercial).
npm WARN deprecated node-uuid@1.4.8: Use uuid module instead
npm WARN deprecated hawk@1.1.1: This module moved to @hapi/hawk. Please make sure to switch over as this distribution is no longer supported and may contain bugs and critical security issues.

added 607 packages, and audited 608 packages in 2m

82 packages are looking for funding
  run `npm fund` for details

16 vulnerabilities (7 moderate, 9 high)

To address all issues (including breaking changes), run:
  npm audit fix --force

Run `npm audit` for details.
@daniel-lewis-ab
Copy link

My fork has 0 vulnerabilities and 25 linting problems. ;)

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

No branches or pull requests

2 participants