Skip to content

Conversation

@jathu
Copy link
Contributor

@jathu jathu commented Jun 4, 2025

Summary

When using tools like Sapling or Jujutsu, these scripts will fail since the .git dir does not exist.

rg "git rev-parse --show-toplevel" --hidden

Test plan

CI

@pytorch-bot
Copy link

pytorch-bot bot commented Jun 4, 2025

🔗 Helpful Links

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

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

❌ 9 New Failures, 48 Pending, 1 Unrelated Failure

As of commit 45c42a4 with merge base 911fb0b (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 Jun 4, 2025
@jathu jathu added the release notes: none Do not include this in the release notes label Jun 4, 2025
@jathu jathu marked this pull request as ready for review June 4, 2025 21:23
@jathu jathu force-pushed the jathu/remove-git-rev-parse branch from 5107abb to 45c42a4 Compare June 4, 2025 21:42
@jathu jathu closed this Jun 4, 2025
@jathu jathu deleted the jathu/remove-git-rev-parse branch June 4, 2025 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/binaries 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: none Do not include this in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants