ci(node): upgrade to node 18

[philibea]

No description provided.

[codecov]

Codecov Report

Merging #732 (3013016) into main (b208776) will not change coverage.
The diff coverage is n/a.

❗ Current head 3013016 differs from pull request most recent head 4bbe086. Consider uploading reports for the commit 4bbe086 to get more accurate results

@@            Coverage Diff            @@
##              main      #732   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           21        20    -1     
  Lines          556       582   +26     
  Branches        78        88   +10     
=========================================
+ Hits           556       582   +26     

Impacted Files	Coverage Δ
packages/use-i18n/src/usei18n.tsx	100.00% <0.00%> (ø)
packages/use-dataloader/src/dataloader.ts	100.00% <0.00%> (ø)
packages/use-dataloader/src/useDataLoader.ts	100.00% <0.00%> (ø)
packages/use-dataloader/src/DataLoaderProvider.tsx	100.00% <0.00%> (ø)
...kages/use-dataloader/src/usePaginatedDataLoader.ts	100.00% <0.00%> (ø)
packages/use-media/src/useMedia.ts

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b208776...4bbe086. Read the comment docs.

[QuiiBz]

Looks like node 18 handles Intl.DateTimeFormat differently compared to node 16/17

[philibea]

Looks like node 18 handles Intl.DateTimeFormat differently compared to node 16/17

@QuiiBz yes it's not the first time we have this issue. We can ignore this actually

[chambo-e]

Looks like node 18 handles Intl.DateTimeFormat differently compared to node 16/17

@QuiiBz yes it's not the first time we have this issue. We can ignore this actually

@philibea behaviour is different in node 14,16,17 and 18, we need to find a way to handle this properly