Skip to content

fix(git): Apply pr hint to git-fetch-with-cli - #17437

Merged
weihanglo merged 1 commit into
rust-lang:masterfrom
epage:pr-git-cli
Sep 3, 2026
Merged

fix(git): Apply pr hint to git-fetch-with-cli#17437
weihanglo merged 1 commit into
rust-lang:masterfrom
epage:pr-git-cli

Conversation

@epage

@epage epage commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

What does this PR try to resolve?

It was added for libgit2 but not git-cli

How to test and review this PR?

I tested this by forcing git-cli for the relevant tests and they still passed.

I did not add any tests for this because we will get coverage with #17329. I avoided duplicating the tests for each git backend out of concern for getting more rate limits from github in CI.

@rustbot rustbot added A-git Area: anything dealing with git S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Sep 3, 2026
@rustbot

rustbot commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

r? @weihanglo

rustbot has assigned @weihanglo.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @epage, @weihanglo
  • @epage, @weihanglo expanded to epage, weihanglo

@weihanglo weihanglo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@rustbot
rustbot added this pull request to the merge queue Sep 3, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Sep 3, 2026
@weihanglo
weihanglo added this pull request to the merge queue Sep 3, 2026
Merged via the queue into rust-lang:master with commit b1e1089 Sep 3, 2026
28 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 3, 2026
@epage
epage deleted the pr-git-cli branch September 3, 2026 20:54
rust-bors Bot pushed a commit to rust-lang/rust that referenced this pull request Sep 5, 2026
Update cargo submodule

9 commits in b2e9d5f9db3fb1c454ab84f10c16508984a266e2..3c0b534756e166d12eb9fd2e1abfe5b42ac6101e
2026-09-02 14:49:16 +0000 to 2026-09-04 17:03:10 +0000
- fix(trim-paths)!: limit options to `none|object|all` (rust-lang/cargo#17432)
- fix(git): Apply pr hint to git-fetch-with-cli (rust-lang/cargo#17437)
- fix(git): Make PR dep note cloer to our style guide (rust-lang/cargo#17436)
- docs: clarify cargo vendor source config option (rust-lang/cargo#17435)
- docs(trim-paths): workspace remap begins with `.` (rust-lang/cargo#17433)
- docs: Switch from "target triple" to "target tuple" (rust-lang/cargo#17430)
- docs(trim-paths): add limitations and polish (rust-lang/cargo#17425)
- fix(git): Simplify error message (rust-lang/cargo#17429)
- chore(deps): update rust crate base64 to 0.23.0 (rust-lang/cargo#17418)
rust-bors Bot pushed a commit to rust-lang/rust that referenced this pull request Sep 5, 2026
Update cargo submodule

9 commits in b2e9d5f9db3fb1c454ab84f10c16508984a266e2..3c0b534756e166d12eb9fd2e1abfe5b42ac6101e
2026-09-02 14:49:16 +0000 to 2026-09-04 17:03:10 +0000
- fix(trim-paths)!: limit options to `none|object|all` (rust-lang/cargo#17432)
- fix(git): Apply pr hint to git-fetch-with-cli (rust-lang/cargo#17437)
- fix(git): Make PR dep note cloer to our style guide (rust-lang/cargo#17436)
- docs: clarify cargo vendor source config option (rust-lang/cargo#17435)
- docs(trim-paths): workspace remap begins with `.` (rust-lang/cargo#17433)
- docs: Switch from "target triple" to "target tuple" (rust-lang/cargo#17430)
- docs(trim-paths): add limitations and polish (rust-lang/cargo#17425)
- fix(git): Simplify error message (rust-lang/cargo#17429)
- chore(deps): update rust crate base64 to 0.23.0 (rust-lang/cargo#17418)
@rustbot rustbot added this to the 1.100.0 milestone Sep 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-git Area: anything dealing with git

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants