Use cimg/node instead of circleci/node

[birkskyum]

As mentioned here:

cimg/node is designed to supercede the legacy CircleCI Node.js image, circleci/node

This is still Node 16, but using the cimg/node as a step towards:

• Use node.js v18 and npm v10 in development #7078

Related to

• Upgrade from Node 16 - it reached end-of-life 11 Sep 2023 #7081

[birkskyum]

Hmm, this didn't see to help. The karma-firefox-launcher is also deprecated, in fact all of karma is, so time is maybe better spent upgrading to something else, than trying to fix it, since a bug-fix wouldn't be merged even if we found an upstream issue.

[archmoj]

I don't see last CircleCI run.
@birkskyum Would you please push to rerun the tests?

[birkskyum]

I don't have permission to manually trigger a run, and i don't know why my commits didn't trigger. I can try again today ofc.

[archmoj]

Please add a newline to draftlogs/7080_change.md file.

[archmoj]

Could you please fix the errors here?
https://app.circleci.com/pipelines/github/plotly/plotly.js?branch=pull%2F7080

[archmoj]

FYI - Some of our mathjax tests started failing after Firefox v82. So we had to add those specific version tests.
See #5993.
It would be nice to investigate if https://bugzilla.mozilla.org/show_bug.cgi?id=1709251 is fixed and see if we could run all tests on most recent Firefox then possibly adjust the config for those tests.

[birkskyum]

It's not fixed in Firefox - had a test break. It's open here https://bugzilla.mozilla.org/show_bug.cgi?id=1350755

[archmoj]

Looking good. Let me re-run the image test.

[archmoj]

@birkskyum Before I merge, are you happy with the updates in this PR or do you want to add additional commits?

[birkskyum]

I'm okay with its current state

[archmoj]

It's not always easy to bump dependencies.
💃
Thanks very much for the PR.