Skip to content

Test database should use real locale configs - #3204

Merged
weiminyu merged 1 commit into
google:masterfrom
weiminyu:assert-db-encoding
Aug 11, 2026
Merged

Test database should use real locale configs#3204
weiminyu merged 1 commit into
google:masterfrom
weiminyu:assert-db-encoding

Conversation

@weiminyu

@weiminyu weiminyu commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

Use the real locale configs from Cloud SQL when creating postgres database using docker.


This change is Reviewable

@weiminyu
weiminyu requested a review from gbrodman August 7, 2026 20:41
@weiminyu
weiminyu force-pushed the assert-db-encoding branch from b6b7911 to 7ca4ed9 Compare August 7, 2026 21:59

@weiminyu weiminyu left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@weiminyu resolved 1 discussion.
Reviewable status: 0 of 2 files reviewed, all discussions resolved (waiting on gbrodman).

@gbrodman gbrodman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gbrodman reviewed 2 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on weiminyu).

Use the real locale configs from Cloud SQL when creating postgres
database using docker.
@weiminyu
weiminyu force-pushed the assert-db-encoding branch from 7ca4ed9 to 628e652 Compare August 11, 2026 15:49
@weiminyu
weiminyu enabled auto-merge August 11, 2026 15:49
@weiminyu
weiminyu added this pull request to the merge queue Aug 11, 2026
Merged via the queue into google:master with commit f408dc0 Aug 11, 2026
16 checks passed
@weiminyu
weiminyu deleted the assert-db-encoding branch August 11, 2026 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants