apache / cordova-ios

Apache Cordova iOS
https://cordova.apache.org/
Apache License 2.0
2.15k stars 987 forks source link

dep!: bump all possible dependencies w/ node engine requirement update >=16.13.0 #1305

Closed erisu closed 1 year ago

erisu commented 1 year ago

Platforms affected

n/a

Motivation and Context

Use latest dependencies and remove any audit warnings in package-lock.json

Description

Bumped:

Other:

Testing

npm t

Checklist

codecov-commenter commented 1 year ago

Codecov Report

Merging #1305 (55e37ec) into master (ad60924) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1305   +/-   ##
=======================================
  Coverage   78.25%   78.25%           
=======================================
  Files          15       15           
  Lines        1761     1761           
=======================================
  Hits         1378     1378           
  Misses        383      383           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

dpogue commented 1 year ago

We're still testing Node 14 in CI (and somehow it appears to be passing?)... presumably we should drop that if we're introducing a node 16.13.0 minimum?

erisu commented 1 year ago

@dpogue yes, you are correct. I have dropped Node 14.x from the CI. Because of the bumping of the packages that increased the node engine requirement, it should also include the CI changes.

breautek commented 1 year ago

Was package-lock rebuilt?

If not we should run npm upgrade to force package-lock to update all of our subdependencies.