npm audit CI change, and minor version bumps#368
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #368 +/- ##
=======================================
Coverage 87.69% 87.69%
=======================================
Files 52 52
Lines 4624 4624
Branches 1307 1307
=======================================
Hits 4055 4055
Misses 354 354
Partials 215 215 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
This looks good. Maybe some things have been updated since you ran |
|
Oh good catch. Some of those are things moving fast - |
|
Updated with the latest. Unfortunately there didn't seem to be any way to use npm-check-updates to only update within a given major version. |
This PR addresses the
npm auditfailures noted in previous PRs, eg:There are 3 parts to this:
npm audit --omit=devto ignore issues reported indevDependencies. Libraries used indevDependenciesare only involved in the dev/build/test process but not included downstream, so software that depends oncql-executionwill not be affected by issues in our devDependencies.npm audit fixoncepackage.jsonby running npm-check-updates . Only patch and minor version updates were included here, not major versions. (For example typescript 6.0.0 is now available, but that upgrade is not included here)cql4browserschanged as a result of the version bumps so is included here.Pull requests into cql-execution require the following.
Submitter and reviewer should ✔ when done.
For items that are not-applicable, mark "N/A" and ✔.
Submitter:
npm run test:plusto run tests, lint, and prettier)cql4browsers.jsbuilt withnpm run build:browserifyif source changed.Reviewer:
Name: