Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

test: Add Node 20 to CI matrix #200

Merged
merged 1 commit into from
Sep 3, 2024
Merged

Conversation

BilalQamar95
Copy link
Contributor

@BilalQamar95 BilalQamar95 commented Aug 21, 2024

Description

As a first step in the upgrade to Node 20, add it to the CI matrix as a non-blocking test.

See the tracking issue for further information.

@BilalQamar95 BilalQamar95 self-assigned this Aug 21, 2024
@coveralls
Copy link

coveralls commented Aug 21, 2024

Pull Request Test Coverage Report for Build 10489654410

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 55.473%

Totals Coverage Status
Change from base Build 10181965677: 0.0%
Covered Lines: 168
Relevant Lines: 305

💛 - Coveralls

Copy link

codecov bot commented Aug 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.22%. Comparing base (f705240) to head (a4c4b9d).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #200   +/-   ##
=======================================
  Coverage   58.22%   58.22%           
=======================================
  Files          25       25           
  Lines         395      395           
  Branches       66       66           
=======================================
  Hits          230      230           
  Misses        149      149           
  Partials       16       16           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@BilalQamar95 BilalQamar95 merged commit 9a1eab6 into master Sep 3, 2024
8 checks passed
@BilalQamar95 BilalQamar95 deleted the bilalqamar95/node20-upgrade-1 branch September 3, 2024 11:43
@brian-smith-tcril brian-smith-tcril mentioned this pull request Sep 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants