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.

Sign up for GitHub

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

Jump to bottom

[browser][icu] Load full ICU in library tests only when it's necessary #82592

Closed
ilonatommy opened this issue Feb 24, 2023 · 2 comments · Fixed by #82619
Closed

[browser][icu] Load full ICU in library tests only when it's necessary #82592

ilonatommy opened this issue Feb 24, 2023 · 2 comments · Fixed by #82619
Assignees
@ilonatommy
Labels
arch-wasm WebAssembly architecture area-Infrastructure-mono

Comments

@ilonatommy
Copy link
Member

Follow-up for #80421.
It's mostly about reverting change in eng/testing/tests.browser.targets made in 63e5e60 to the state of: 87caebd and making sure it's working fine.
@ilonatommy ilonatommy self-assigned this Feb 24, 2023
@ghost ghost added the untriaged New issue has not been triaged by the area owner label Feb 24, 2023
@ghost
Copy link

ghost commented Feb 24, 2023

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details

Follow-up for #80421.
It's mostly about reverting change in eng/testing/tests.browser.targets made in 63e5e60 to the state of: 87caebd and making sure it's working fine.

Author: ilonatommy
Assignees: ilonatommy
Labels:

arch-wasm, area-Infrastructure-libraries
Milestone: -

@ilonatommy ilonatommy mentioned this issue Feb 24, 2023
Merged
@ilonatommy ilonatommy removed the untriaged New issue has not been triaged by the area owner label Feb 24, 2023
@ilonatommy ilonatommy mentioned this issue Feb 24, 2023
Merged
@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Feb 24, 2023
@ghost
Copy link

ghost commented Feb 27, 2023

Tagging subscribers to this area: @directhex
See info in area-owners.md if you want to be subscribed.

Issue Details

Follow-up for #80421.
It's mostly about reverting change in eng/testing/tests.browser.targets made in 63e5e60 to the state of: 87caebd and making sure it's working fine.

Author: ilonatommy
Assignees: ilonatommy
Labels:

arch-wasm, area-Infrastructure-mono, in-pr
Milestone: -

@ghost ghost removed the in-pr There is an active PR which will close this issue when it is merged label Mar 1, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Mar 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ilonatommy ilonatommy

Labels
arch-wasm WebAssembly architecture area-Infrastructure-mono
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[browser][icu][libraries] Load full ICU in library tests only when it's necessary ilonatommy/runtime
2 participants
@carlossanlop @ilonatommy