Skip to content

Correct the conformance polyglot command - #146

Merged
rmcdaniel merged 1 commit into
mainfrom
docs/polyglot-runbook-command
Sep 30, 2026
Merged

rmcdaniel merged 1 commit into
mainfrom
docs/polyglot-runbook-command

Conversation

@rmcdaniel

Copy link
Copy Markdown
Member

Fix

Correct the public conformance runbook's Polyglot command. Sample App provides scripts/polyglot.sh. The listed scripts/polyglot-validation.sh does not exist.

Follow-up to #122's executable-instruction audit. This patch corrects the command and retains the experiment's scope.

Verification

Checked Sample App's current main file inventory and its polyglot validation workflow, which invokes scripts/polyglot.sh. The latest merged-main published-artifact smoke passes with CLI 2.1.3, Server 2.4.31 and the frozen PHP/Python/Rust SDK tuple. Normal shared repository checks are required for this documentation change. No prose-mirroring test is added.

@rmcdaniel

Copy link
Copy Markdown
Member Author

Reviewed the complete one-line diff at 1b87990cd994c4624f8b0d3d2a731c7cd57edc1b. The corrected command exists in Sample App's current main and is invoked by its ordinary published PHP/Python/Rust smoke workflow. The latest exact-consumer smoke passes in run 36719012982. This patch changes the executable instruction without changing scenario scope or adding a compatibility claim. Merge after normal Shared contract tests pass, then verify the public document and remote-branch deletion.

@rmcdaniel
rmcdaniel marked this pull request as ready for review September 30, 2026 13:54
@rmcdaniel
rmcdaniel merged commit e221798 into main Sep 30, 2026
1 check passed
@rmcdaniel
rmcdaniel deleted the docs/polyglot-runbook-command branch September 30, 2026 13: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.

2 participants