Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #175 +/- ##
==========================================
- Coverage 53.84% 50.60% -3.25%
==========================================
Files 4 5 +1
Lines 78 83 +5
==========================================
Hits 42 42
- Misses 36 41 +5 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
I suggest flagging this as a breaking change and updating the Node.js engine requirements to: "engines": {
"node": ">=20.17.0 || >=22.9.0"
}The current Node.js engine requirement is not compatible with It may be worth reviewing all dependencies and sub-dependencies to confirm whether |
Yah I didn't think to actually check the engines and ensure they still match up. Will try to do it sometime this week. |
Platforms affected
Motivation and Context
Solves all audit issues
Description
Updated all packages to the latest versions.
Ran npm upgrade to also update all subdependencies.
Updated eslint, copied the config from another cordova repo, adjusted slightly for what exists in this plugman repo.
p-tryandp-each-seriesI left as is as I have plans to remove these dependencies completely.Testing
Ran npm test
Checklist
(platform)if this change only applies to one platform (e.g.(android))