Skip to content

Add missing types to RunBuildOptions and export RunBuildResult#1621

Closed
pafry7 wants to merge 2 commits intofacebook:mainfrom
pafry7:fix/runbuildoptions-assets-type
Closed

Add missing types to RunBuildOptions and export RunBuildResult#1621
pafry7 wants to merge 2 commits intofacebook:mainfrom
pafry7:fix/runbuildoptions-assets-type

Conversation

@pafry7
Copy link
Copy Markdown
Contributor

@pafry7 pafry7 commented Nov 21, 2025

Summary

This commit added the assets boolean to RunBuildOptions as well as RunBuildResult type, but index.d.ts was not changed, which causes missing type errors.

Screenshot 2025-11-21 at 08 15 41

Changelog: [Fix] missing types for runBuild command

Test plan

Screenshot 2025-11-21 at 08 23 03 Screenshot 2025-11-21 at 08 23 15

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 21, 2025
@facebook-github-bot facebook-github-bot added the Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. label Nov 21, 2025
@pafry7 pafry7 force-pushed the fix/runbuildoptions-assets-type branch from fe0f613 to 144f4a5 Compare November 21, 2025 07:46
@pafry7 pafry7 changed the title Add missing assets to RunBuildOption interface Add missing types to RunBuildOptions and export runBuildResult Nov 21, 2025
@pafry7 pafry7 changed the title Add missing types to RunBuildOptions and export runBuildResult Add missing types to RunBuildOptions and export RunBuildResult Nov 21, 2025
@pafry7 pafry7 force-pushed the fix/runbuildoptions-assets-type branch from 144f4a5 to 1ac9718 Compare November 21, 2025 07:52
@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync bot commented Nov 21, 2025

@robhogan has imported this pull request. If you are a Meta employee, you can view this in D87629698.

@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync bot commented Nov 24, 2025

@robhogan merged this pull request in 3266dd6.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants