chore(deps): Update some dependencies, add Node 20 to CI

apache / cordova-lib

Apache Cordova Tooling Library

https://cordova.apache.org/

Apache License 2.0

221 stars 242 forks source link

chore(deps): Update some dependencies, add Node 20 to CI #924

Closed dpogue closed 5 months ago

dpogue commented 5 months ago

Platforms affected

All

Motivation and Context

Keep things up to date and resolve some npm audit warnings about dependencies.

Description

Rebuild package-lock.json with updated dependencies
Drop pify, delay, and fs-extra dependencies
Migrate from fs-extra APIs to built-in NodeJS fs APIs
Minor cleanups of some CodeQL-identified syntax errors in dummy test files
Update GitHub Actions steps to latest versions
Add CodeQL scanning as part of CI
Add Node 20 to CI matrix

Testing

All unit and integration tests pass locally and in CI

Checklist

[x] I've run the tests to see all new and existing tests pass

github-advanced-security[bot] commented 5 months ago

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

codecov-commenter commented 5 months ago

Codecov Report

Attention: Patch coverage is 91.95402% with 7 lines in your changes are missing coverage. Please review.

Project coverage is 87.92%. Comparing base (66c9670) to head (90da6e5).

Files	Patch %	Lines
src/plugman/init-defaults.js	0.00%	3 Missing :warning:
src/plugman/platform.js	50.00%	2 Missing :warning:
src/plugman/fetch.js	92.85%	1 Missing :warning:
src/plugman/install.js	85.71%	1 Missing :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #924 +/- ## ======================================= Coverage 87.92% 87.92% ======================================= Files 46 46 Lines 2129 2129 ======================================= Hits 1872 1872 Misses 257 257 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.