Skip to content

Conversation

@8Keep
Copy link
Contributor

@8Keep 8Keep commented Dec 17, 2025

Summary:
Cleaning up platform args before the whole feature gets deleted. These missed our validation.


These are to force android builds of mwa for xplat diffs

Reviewed By: mzlee

Differential Revision: D89322661

@pytorch-bot
Copy link

pytorch-bot bot commented Dec 17, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/16299

Note: Links to docs will display an error until the docs builds have been completed.

✅ You can merge normally! (1 Unrelated Failure)

As of commit ea20160 with merge base 257da59 (image):

UNSTABLE - The following job is marked as unstable, possibly due to flakiness on trunk:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@meta-codesync
Copy link

meta-codesync bot commented Dec 17, 2025

@8Keep has exported this pull request. If you are a Meta employee, you can view the originating Diff in D89322661.

@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 Dec 17, 2025
@8Keep
Copy link
Contributor Author

8Keep commented Dec 17, 2025

@pytorchbot label "release notes: none"

@pytorch-bot pytorch-bot bot added the release notes: none Do not include this in the release notes label Dec 17, 2025
…er hit validation (pytorch#16299)

Summary:

Cleaning up platform args before the whole feature gets deleted. These missed our validation.

---
These are to force android builds of mwa for xplat diffs

Reviewed By: mzlee

Differential Revision: D89322661
Copy link
Member

@GregoryComer GregoryComer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. I'm not an expert on buck, but as long as OSS CI is green, we should be good. You will need to ensure that the internal diff is in sync with the PR, which can be a pain with internal-only changes.

meta-codesync bot pushed a commit to facebook/ocamlrep that referenced this pull request Dec 17, 2025
…er hit validation

Summary:
X-link: pytorch/executorch#16299

Cleaning up platform args before the whole feature gets deleted. These missed our validation.

 ---
These are to force android builds of mwa for xplat diffs

Reviewed By: mzlee, GregoryComer

Differential Revision: D89322661

fbshipit-source-id: d777f02d86b9118f300c92a572755f97e6811400
meta-codesync bot pushed a commit to facebook/buck2 that referenced this pull request Dec 17, 2025
…er hit validation

Summary:
X-link: pytorch/executorch#16299

Cleaning up platform args before the whole feature gets deleted. These missed our validation.

 ---
These are to force android builds of mwa for xplat diffs

Reviewed By: mzlee, GregoryComer

Differential Revision: D89322661

fbshipit-source-id: d777f02d86b9118f300c92a572755f97e6811400
meta-codesync bot pushed a commit to facebook/buck2-shims-meta that referenced this pull request Dec 17, 2025
…er hit validation

Summary:
X-link: pytorch/executorch#16299

Cleaning up platform args before the whole feature gets deleted. These missed our validation.

 ---
These are to force android builds of mwa for xplat diffs

Reviewed By: mzlee, GregoryComer

Differential Revision: D89322661

fbshipit-source-id: d777f02d86b9118f300c92a572755f97e6811400
@meta-codesync meta-codesync bot merged commit 12ace93 into pytorch:main Dec 17, 2025
137 of 141 checks passed
xingguo01 pushed a commit to xingguo01/executorch that referenced this pull request Dec 18, 2025
…er hit validation

Differential Revision: D89322661

Pull Request resolved: pytorch#16299
jirioc pushed a commit to nxp-upstream/executorch that referenced this pull request Dec 19, 2025
…er hit validation

Differential Revision: D89322661

Pull Request resolved: pytorch#16299
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. fb-exported meta-exported release notes: none Do not include this in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants