Hi maintainers 👋 I ran SonarQube in my IDE and found 7 issues in src/main.js.
I’m reporting for visibility only (no fix PR for these), since my current PR is scoped to the icon path correction.
Summary
- File:
src/main.js
- Total issues:
7
- Rules involved:
javascript:S7772, javascript:S2486, javascript:S3735, javascript:S7785, javascript:S6557
Findings
-
javascript:S7772 – Prefer node:path over path
Location: (2,17)
-
javascript:S7772 – Prefer node:url over url
Location: (4,30)
-
javascript:S7772 – Prefer node:fs over fs
Location: (6,15)
-
javascript:S2486 – Handle this exception or don’t catch it at all
Location: (37,6)
-
javascript:S3735 – Unexpected negated condition
Location: (89,4)
-
javascript:S7785 – Prefer top-level await over using a promise chain
Location: (102,20)
-
javascript:S6557 – Use String#startsWith method instead
Location: (125,12)
Hi maintainers 👋 I ran SonarQube in my IDE and found 7 issues in
src/main.js.I’m reporting for visibility only (no fix PR for these), since my current PR is scoped to the icon path correction.
Summary
src/main.js7javascript:S7772,javascript:S2486,javascript:S3735,javascript:S7785,javascript:S6557Findings
javascript:S7772 – Prefer
node:pathoverpathLocation:
(2,17)javascript:S7772 – Prefer
node:urloverurlLocation:
(4,30)javascript:S7772 – Prefer
node:fsoverfsLocation:
(6,15)javascript:S2486 – Handle this exception or don’t catch it at all
Location:
(37,6)javascript:S3735 – Unexpected negated condition
Location:
(89,4)javascript:S7785 – Prefer top-level
awaitover using a promise chainLocation:
(102,20)javascript:S6557 – Use
String#startsWithmethod insteadLocation:
(125,12)