Skip to content

Conversation

@jackzhxng
Copy link
Contributor

Use recursive submodule so that submodules of submodules (and on) are initialized, namely tokenizers's submodules.

@pytorch-bot
Copy link

pytorch-bot bot commented Apr 25, 2025

🔗 Helpful Links

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

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

❌ 2 New Failures, 1 Pending, 1 Unrelated Failure

As of commit a28bcc9 with merge base 2837867 (image):

NEW FAILURES - The following jobs have failed:

FLAKY - The following job failed but was likely due to flakiness present on trunk:

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

@facebook-github-bot facebook-github-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 Apr 25, 2025
@jackzhxng jackzhxng changed the title Recursive checkout Recursive checkout in CI Apr 25, 2025
@jackzhxng jackzhxng temporarily deployed to upload-benchmark-results April 25, 2025 09:20 — with GitHub Actions Inactive
@jackzhxng jackzhxng temporarily deployed to upload-benchmark-results April 25, 2025 09:20 — with GitHub Actions Inactive
@jackzhxng jackzhxng added the release notes: build Changes related to build, including dependency upgrades, build flags, optimizations, etc. label Apr 25, 2025
@jackzhxng jackzhxng merged commit 7e034ca into main Apr 25, 2025
344 of 349 checks passed
@jackzhxng jackzhxng deleted the jz/recursive-co branch April 25, 2025 17:17
@jackzhxng jackzhxng mentioned this pull request Apr 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. release notes: build Changes related to build, including dependency upgrades, build flags, optimizations, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants