Repository navigation
[Scheduled Run] Localized resource files from OneLocBuild - #4612
Closed
cheenamalhotra wants to merge 1 commit into
Closed
cheenamalhotra wants to merge 1 commit into
cheenamalhotra wants to merge 1 commit into
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Localizes two server-certificate error messages across 13 resource files.
Changes:
- Translates certificate loading/parsing errors.
- Translates missing server certificate errors.
Reviewed changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
Strings.zh-Hant.resx |
Traditional Chinese translations |
Strings.zh-Hans.resx |
Simplified Chinese translations |
Strings.tr.resx |
Turkish translations |
Strings.ru.resx |
Russian translations |
Strings.pt-BR.resx |
Brazilian Portuguese translations |
Strings.pl.resx |
Polish translations |
Strings.ko.resx |
Korean translations |
Strings.ja.resx |
Japanese translations |
Strings.it.resx |
Italian translations; one quotation error found |
Strings.fr.resx |
French translations |
Strings.es.resx |
Spanish translations |
Strings.de.resx |
German translations |
Strings.cs.resx |
Czech translations |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| </data> | ||
| <data name="SQL_ServerCertificateFileLoadFailed" xml:space="preserve"> | ||
| <value>The certificate file specified by the 'ServerCertificate' option could not be loaded or parsed: '{0}'.</value> | ||
| <value>Non è possibile caricare o analizzare il file del certificato specificato dall'opzione 'ServerCertificate: '{0}'.</value> |
This was referenced Aug 31, 2026
priyankatiwari08
left a comment
priyankatiwari08
left a comment
Contributor
There was a problem hiding this comment.
Summary
Scheduled OneLocBuild run translating SQL_ServerCertificateFileLoadFailed and SQL_ServerCertificateNotAvailable across the 13 localized Strings.*.resx files (+26/-26). No code, no public API, no behavioral change.
Assessment
The change itself is mechanically correct — same two resource keys, same 13 files, existing entries untouched.
However, this PR appears to be superseded.
PRs #4612, #4613 and #4614 are byte-for-byte identical changes against the same base (main), generated by consecutive daily scheduled runs (2026-08-28, 08-29, 08-30). Only the newest — #4614 — should be merged; merging this one as well produces a redundant/conflicting merge. Recommend closing this PR in favour of #4614.
Separately, it would be worth having the scheduled job close its own superseded PRs so duplicates don't accumulate in the review queue.
Minor content observation (applies to all three)
Strings.it.resx: the ItalianSQL_ServerCertificateFileLoadFailedvalue readsl'opzione 'ServerCertificate: '{0}'.— the quoting around the option name looks unbalanced. Worth a loc-team confirmation; not a build or runtime issue.
No correctness, security, concurrency or API concerns.
priyankatiwari08
added a commit
that referenced
this pull request
Sep 2, 2026
* Add idempotent localization PR script to stop duplicate PRs The scheduled Localization-CI pipeline opens a brand-new GitHub PR on every run because its inline "Open PR on GitHub" step pushes a timestamped branch (dev/automation/onelocbuild-<yyyyMMdd-HHmmss>) and never checks whether an equivalent PR is already open. Four byte-for-byte identical PRs (#4607, #4612, #4613, #4614) accumulated as a result. Add eng/pipelines/scripts/Open-LocalizationPr.ps1 as a reusable, idempotent replacement for that inline step: - Uses a stable branch name, rebuilt from the base branch each run, so no timestamped branch proliferation and no commit accumulation. - Exits without pushing or calling the GitHub API when the localized resources are identical to the base branch. - Skips the force-push when the remote branch already holds the exact same tree on top of the same base. - Reuses an already-open pull request (PATCH) instead of opening a second one, and otherwise opens exactly one new PR. Includes Pester v5 tests covering the de-duplication contract with git and Invoke-RestMethod mocked. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: cb6c5b07-fb77-43bc-a4aa-a9d39aa04438 * Add -DryRun switch to Open-LocalizationPr.ps1 Allows validating the pipeline wiring (paths, token scopes, OneLocBuild output, existing-PR lookup) from a feature branch without pushing a branch or creating/updating a pull request in the public repository. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: cb6c5b07-fb77-43bc-a4aa-a9d39aa04438 * Address review feedback on Open-LocalizationPr - Authenticate git via GIT_CONFIG_* environment config instead of embedding the token in the remote URL, so it never reaches .git/config, a process command line, or Invoke-Git exception messages. Cleared in the finally block. - Read GitHub error bodies from $_.ErrorDetails first, since PowerShell 7 exposes an HttpResponseMessage with no GetResponseStream(); keep the stream path as a Windows PowerShell fallback so 4xx bodies are no longer dropped. - Replace the unconditional force-push with --force-with-lease pinned to the remote SHA observed earlier in the run, so an overlapping run fails instead of discarding a concurrent localization result. Adds 4 tests (17 total, all passing). Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: cb6c5b07-fb77-43bc-a4aa-a9d39aa04438 --------- Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: cb6c5b07-fb77-43bc-a4aa-a9d39aa04438
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated PR created from the OneLocBuild scheduled pipeline run.
Contains updated localized
Strings.*.resxresource files.