Skip to content

tweak some git clone commands #2497

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

tweak some git clone commands #2497

wants to merge 1 commit into from

Conversation

tshepang
Copy link
Member

@tshepang tshepang commented Jul 9, 2025

  • --depth=1 is more useful for once-off uses, like on ci
  • .git postfix on github repo url is not needed

- --depth=1 is more useful for once-off uses, like on ci
- .git postfix on github repo url is not needed
@rustbot
Copy link
Collaborator

rustbot commented Jul 9, 2025

Thanks for the PR. If you have write access, feel free to merge this PR if it does not need reviews. You can request a review using r? rustc-dev-guide or r? <username>.

@rustbot rustbot added the S-waiting-on-review Status: this PR is waiting for a reviewer to verify its content label Jul 9, 2025
@jieyouxu
Copy link
Member

jieyouxu commented Jul 9, 2025

--depth=1 can be intentional sometimes, if history doesn't matter for your particular workflow instance.
r? @ZuseZ4 (autodiff workflow, I'm not sure)

@Noratrieb
Copy link
Member

wasn't there a problem that depth 1 doesn't work properly with bootstrap?

@jieyouxu
Copy link
Member

jieyouxu commented Jul 9, 2025

For rust-lang/rust almost certainly, I was moreso wondering about the enzyme/llvm cases

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: this PR is waiting for a reviewer to verify its content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants