zooltd commented on pull request #51: URL: https://github.com/apache/skywalking-eyes/pull/51#issuecomment-891544934
> This looks much better, just some nits. Can you please run this tool on [http://github.com/apache/skywalking-client-js](https://github.com/apache/skywalking-client-js) and [http://github.com/apache/skywalking-nodejs](https://github.com/apache/skywalking-nodejs) to see the results and paste here? For skywalking-client-js, the total output is shown as below: root@Samaritan:~/skywalking-eyes# ./bin/linux/license-eye -c ~/testPlace/skywalking-client-js/.licenserc.yaml d r INFO GITHUB_TOKEN is not set, license-eye won't comment on the pull request INFO Loading configuration from file: /root/testPlace/skywalking-client-js/.licenserc.yaml INFO Try to install nodejs packages in 5 seconds, press [s/S] and ENTER to skip INFO Time out, try to install packages INFO Run command: /usr/local/bin/npm install, please wait npm WARN old lockfile npm WARN old lockfile The package-lock.json file was created with an old version of npm, npm WARN old lockfile so supplemental metadata must be fetched from the registry. npm WARN old lockfile npm WARN old lockfile This is a one-time fix-up, please be patient... npm WARN old lockfile npm WARN deprecated [email protected]: Please see https://github.com/lydell/urix#deprecated npm WARN deprecated [email protected]: https://github.com/lydell/resolve-url#deprecated npm WARN deprecated [email protected]: Please upgrade to version 7 or higher. Older versions may use Math.random() in certain circumstances, which is known to be problematic. See https://v8.dev/blog/math-random for details. npm WARN deprecated [email protected]: The querystring API is considered Legacy. new code should use the URLSearchParams API instead. npm WARN deprecated [email protected]: Chokidar 2 will break on node v14+. Upgrade to chokidar 3 with 15x less dependencies. added 652 packages, and audited 653 packages in 29s 50 packages are looking for funding run `npm fund` for details 5 vulnerabilities (4 moderate, 1 high) To address issues that do not require attention, run: npm audit fix To address all issues (including breaking changes), run: npm audit fix --force Run `npm audit` for details. Dependency | License --------- | ------------ js-base64 | BSD-3-Clause -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
