Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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 join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug 🐛 Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant