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

Revert locale changes #42583

Closed
wants to merge 7 commits into from
Closed

Revert locale changes #42583

wants to merge 7 commits into from

Conversation

alxhub
Copy link
Member

@alxhub alxhub commented Jun 16, 2021

These commits caused issues in g3.

alxhub added 7 commits June 16, 2021 09:23
@alxhub
Revert "build: generate alias locale data for closure locale (angular…
c01a81d 
…#42230)"

This reverts commit 044e022.
@alxhub
Revert "build: simplify generation of closure locale file (angular#42230
17887dd 
)"

This reverts commit 8f24d71.
@alxhub
Revert "build: add documentation for generate-locales-tool (angular…
a4df0ba 
…#42230)"

This reverts commit d4c880b.
@alxhub
Revert "build: build locale files for legacy saucelabs job (angular#4…
9323459 
…2230)"

This reverts commit c42041b.
@alxhub
Revert "build: wire up new CLDR generation tool within Bazel (angular…
d6e37d3 
…#42230)"

This reverts commit 4957da8.
@alxhub
Revert "build: convert CLDR locale extraction from Gulp to Bazel tool (…
6a5043c 
…angular#42230)"

This reverts commit 1eaeb23.
@alxhub
Revert "refactor: remove checked-in locale files (angular#42230)"
5051bad 
This reverts commit 5822771.
@google-cla google-cla bot added the cla: yes label Jun 16, 2021
@pullapprove pullapprove bot requested review from AndrewKushnir and IgorMinar June 16, 2021 16:27
@alxhub alxhub added action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release labels Jun 16, 2021
@alxhub alxhub closed this in d6cca3c Jun 16, 2021
alxhub added a commit that referenced this pull request Jun 16, 2021
@alxhub
Revert "build: simplify generation of closure locale file (#42230)" (#…
2cd1c6c 
…42583)

This reverts commit 8f24d71.

PR Close #42583
alxhub added a commit that referenced this pull request Jun 16, 2021
@alxhub
Revert "build: add documentation for generate-locales-tool (#42230)" (
76484f9 
#42583)

This reverts commit d4c880b.

PR Close #42583
alxhub added a commit that referenced this pull request Jun 16, 2021
@alxhub
Revert "build: build locale files for legacy saucelabs job (#42230)" (#…
bf0e82c 
…42583)

This reverts commit c42041b.

PR Close #42583
alxhub added a commit that referenced this pull request Jun 16, 2021
@alxhub
Revert "build: wire up new CLDR generation tool within Bazel (#42230)" (
877cde8 
#42583)

This reverts commit 4957da8.

PR Close #42583
alxhub added a commit that referenced this pull request Jun 16, 2021
@alxhub
Revert "build: convert CLDR locale extraction from Gulp to Bazel tool (…
ec6dc78 
…#42230)" (#42583)

This reverts commit 1eaeb23.

PR Close #42583
alxhub added a commit that referenced this pull request Jun 16, 2021
@alxhub
Revert "refactor: remove checked-in locale files (#42230)" (#42583)
2d1347b 
This reverts commit 5822771.

PR Close #42583
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Jul 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@AndrewKushnir AndrewKushnir AndrewKushnir approved these changes

@IgorMinar IgorMinar Awaiting requested review from IgorMinar

Assignees
No one assigned
Labels
action: merge The PR is ready for merge by the caretaker cla: yes target: minor This PR is targeted for the next minor release
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@alxhub @AndrewKushnir