Closed carolabadeer closed 1 year ago
Merging #574 (ccaab4f) into master (a476e2b) will not change coverage. The diff coverage is
n/a
.:exclamation: Current head ccaab4f differs from pull request most recent head cb4e545. Consider uploading reports for the commit cb4e545 to get more accurate results
@@ Coverage Diff @@
## master #574 +/- ##
=======================================
Coverage 83.38% 83.38%
=======================================
Files 37 37
Lines 1794 1794
=======================================
Hits 1496 1496
Misses 298 298
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
Issue #, if available: N/A
Description of changes: PR build for Node 18.x started failing due to the latest version of
npm
not being used in the "Setup Node 18.x" step.Updating
actions/checkout
andactions/setup-node
to v3 and adding thecheck-latest: true
flag to ensure the latest npm version is being used resolved this error.Codecov
package used in "Report Coverage" step was also reporting aTypeError
that was fixed inv3.8.3
, see related issue.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.