Skip to content

Conversation

@clydin
Copy link
Member

@clydin clydin commented Nov 17, 2025

Reorders the type union in the runnerConfig option's schema to prioritize string over boolean.

This change provides a stronger hint to the CLI's argument parser, ensuring that when a value is provided for the option (e.g., a file path), it is unambiguously treated as a string. This improves the robustness of the parsing logic for this union-type option while preserving the functionality of its boolean flag forms.

@clydin clydin added the target: rc This PR is targeted for the next release-candidate label Nov 17, 2025
@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Nov 17, 2025
Reorders the `type` union in the `runnerConfig` option's schema to
prioritize `string` over `boolean`.

This change provides a stronger hint to the CLI's argument parser,
ensuring that when a value is provided for the option (e.g., a file path),
it is unambiguously treated as a string. This improves the robustness of
the parsing logic for this union-type option while preserving the
functionality of its boolean flag forms.
@clydin clydin force-pushed the unit-test/runner-config-path-cmd branch from 4da9400 to fd316cf Compare November 17, 2025 17:48
@dgp1130 dgp1130 merged commit ed700a1 into angular:main Nov 17, 2025
31 checks passed
@dgp1130
Copy link
Collaborator

dgp1130 commented Nov 17, 2025

This PR was merged into the repository. The changes were merged into the following branches:

@clydin clydin deleted the unit-test/runner-config-path-cmd branch November 17, 2025 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: @angular/build target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants