test: Add Node 20 to CI matrix

[BilalQamar95]

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.

[UsamaSadiq]

Let's mention why this change is needed.

[BilalQamar95]

The error (A control must be associated with a text label) and the corresponding ESLint ignore might seem self-explanatory as the rule jsx-a11y/control-has-associated-label typically applies to interactive controls like buttons, inputs, or other form elements that require labels for accessibility. In this case, <td> is a table cell element, which doesn’t function as an interactive control and therefore doesn’t require a label. The eslint-disable comment was added to prevent unnecessary warnings for this specific use case.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.77%. Comparing base (a681333) to head (3939b5a).

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1445   +/-   ##
=======================================
  Coverage   87.77%   87.77%           
=======================================
  Files         310      310           
  Lines        5319     5319           
  Branches     1315     1315           
=======================================
  Hits         4669     4669           
  Misses        633      633           
  Partials       17       17           

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