Skip to content

[stable35] fix(openapi): propagate TypeScript generation failures (stable35 backport) - #9462

Merged
vitormattos merged 1 commit into
stable35from
backport/9457/stable35
Nov 13, 2026
Merged

vitormattos merged 1 commit into
stable35from
backport/9457/stable35

Conversation

@vitormattos

Copy link
Copy Markdown
Member

Backport of #9457 to stable35

Restores correct exit status handling for the OpenAPI/TypeScript generation command and provides an OpenAPI-only alternative:

Stable branch adaptation

The tests/php/openapi/phpunit.xml test suite introduced on main does not exist on stable35. Place OpenApiGenerationTest.php under tests/php/Unit/ instead, so the pre-existing unit suite discovers it. No change to the stable PHPUnit configuration is needed.

Verification

  • Compared branch with stable35: only composer.json and tests/php/Unit/OpenApiGenerationTest.php changed.
  • PHP 8.4 CLI synthetic generator checks: 6/6 passed, including errors 23, 37, 127 and success paths.
  • Commit created with verified OpenPGP signature and matching DCO sign-off.
  • Not run locally: actual PHPUnit suite or full OpenAPI/TypeScript regeneration; CI needs to validate integration in the target branch.

Backport source commit: 83949bb0a69e9bbe12c109aa0618712a17aec7b3.

No automatic merge requested.

Backport of #9457. Adapt regression tests to the existing PHPUnit unit suite.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos
vitormattos requested a review from a team as a code owner November 13, 2026 16:29
@github-actions github-actions Bot changed the title fix(openapi): propagate TypeScript generation failures (stable35 backport) [stable35] fix(openapi): propagate TypeScript generation failures (stable35 backport) Nov 13, 2026
@vitormattos vitormattos added this to the Next Patch (35) milestone Nov 13, 2026
@vitormattos
vitormattos merged commit 399184c into stable35 Nov 13, 2026
63 checks passed
@vitormattos
vitormattos deleted the backport/9457/stable35 branch November 13, 2026 16:48
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.

1 participant