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
Fix i18n file diff test #7187
Fix i18n file diff test #7187
Conversation
58f1c35
to
6f5e0e5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Thanks!
Frontend test is going to fail until #7185 merges. |
Looks like it's working :) /hold until #7185 merges to fix the JSON files |
6f5e0e5
to
90c6b9d
Compare
Rebasing. |
90c6b9d
to
b7cb169
Compare
Another JSON file came in; updated to see if frontend tests will now pass. |
b7cb169
to
dbeb595
Compare
Another JSON file wasn't checked in. |
dbeb595
to
7dd3944
Compare
6f95f01
to
19b656a
Compare
One of the i18n files that hadn't been checked in was conflicting with a namespace in another package. I updated the namespace that was smaller to fix the e2e tests. Long term, we should have a tool to check for these. Can you review this @spadgett assuming tests now pass? Thanks. |
@spadgett - I updated file locations and imports; still have an open query on the support locales object. |
/test e2e-gcp-console |
/retest |
241784b
to
b654715
Compare
14cd82f
to
de92287
Compare
Test was still using the old file location for i18n files. I updated it to handle public and packages.
de92287
to
473ed7a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rebeccaalpert, spadgett The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/hold cancel |
/retest |
/retest Please review the full test history for this PR and help us cut down flakes. |
1 similar comment
/retest Please review the full test history for this PR and help us cut down flakes. |
The diff test for i18n files was still using the old file location (
./locales
). I updated it to handle public and packages locales folders.