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

Internationalization of Lighthouse (Phase 1) #5719

Closed
19 of 24 tasks
paulirish opened this issue Jul 24, 2018 · 3 comments
Closed
19 of 24 tasks

Internationalization of Lighthouse (Phase 1) #5719

paulirish opened this issue Jul 24, 2018 · 3 comments
Assignees
Labels

Comments

@paulirish
Copy link
Member

paulirish commented Jul 24, 2018

Through a series of PRs and CLs we're much closer to having Lighthouse i18n-ready than before. Some relevant PRs: #5644, #5655, #5713, #5716, #5718

Known remaining work:

ship l10n'd lighthouse to extension/devtools

Plz add anything else here

@brendankenny
Copy link
Member

figure out Node's ICU support on Travis/Appveyor (Number.toLocaleString appears to always use 'en-US')

can we resolve with an NVM flag or travis config?

looks like yes we can, but have to pass --with-intl=full-icu to nvm manually instead of relying on travis's built-in version selection.

See travis-ci/travis-ci#9422 (comment) and nvm-sh/nvm#1719.

I haven't looked into appveyor yet.

@thom4parisot
Copy link

Newest release of nvm should address #5719 (comment)

@paulirish
Copy link
Member Author

We call this one done, folks. 👍 👍 👍 👍 👍

#7238 "Internationalization of Lighthouse (Phase 2)" paints the road ahead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants