prantlf / jsonlint

JSON/CJSON/JSON5 parser, syntax & schema validator and pretty-printer with a command-line client, written in pure JavaScript.
http://prantlf.github.io/jsonlint/
MIT License
38 stars 9 forks source link

fix:v0.0.1 bumping deps as old deps had vulnerabilities #13

Closed ghost closed 2 years ago

ghost commented 2 years ago

not sure if this repo is maintained anymore, but since it was more current than it's parent, I went and updated the dependencies, as one had a vulnerability.

I did my best to only touch the minimum required to get the unit tests working:

npm i;
npm run preflight;
npm test;

preflight does: npm update;npm run npm-check-updates;npm i;npm audit fix;npm outdated;npm run lint;

npm-check-updates just checks for updates for all modules and applies them.

codecov[bot] commented 2 years ago

Codecov Report

Merging #13 (5f07cb5) into master (1a7c5ba) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #13   +/-   ##
=======================================
  Coverage   87.21%   87.21%           
=======================================
  Files           5        5           
  Lines         391      391           
=======================================
  Hits          341      341           
  Misses         50       50           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 1a7c5ba...5f07cb5. Read the comment docs.

prantlf commented 2 years ago

Thanks for the help! Although in this case more reminding me. I'm sorry, I didn't merge your change. I made a big overhaul to the dependencies by 0d35969. Partially with ncu -u, but also by replacing some tools, which are not maintained any more.