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

Latest Node 14 + 16 breaking tests/CI #2169

Closed
notjosh opened this issue Feb 10, 2022 · 0 comments · Fixed by #2171
Closed

Latest Node 14 + 16 breaking tests/CI #2169

notjosh opened this issue Feb 10, 2022 · 0 comments · Fixed by #2171
Labels
Type: Bug 🐛 Something isn't working

Comments

@notjosh
Copy link
Contributor

notjosh commented Feb 10, 2022

Overview

Node v14.19.0 and v16.14.0 were released in the last week or two, and are making their way to the CI runners.

Both of them include a bump to ICU (nodejs/node#40658), which is breaking some i18n related tests (locally and on CI). Note: CI still uses the previous Node 16 release, so only Node 14 is a problem there.

We can see on CI, two builds on the same commit from yesterday and today:

This is reproducible locally also:

$ nvm install 14          # or 16
$ nvm use 14              # or 16
Now using node v14.19.0   # v16.14.0
$ yarn
$ yarn test packages/react-i18n/src/tests/i18n.test.ts

I believe it's calling into @shopify/dates for the formatting, but the failing tests are in @shopify/react-i18n.

@notjosh notjosh added the Type: Bug 🐛 Something isn't working label Feb 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Type: Bug 🐛 Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant