Skip to content
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

[beta-1.79] fix(config): Ensure --config net.git-fetch-with-cli=true is respected #13997

Merged
merged 3 commits into from
Jun 3, 2024

Conversation

epage
Copy link
Contributor

@epage epage commented Jun 1, 2024

epage added 2 commits May 31, 2024 20:30
This reverts commit f525e1f.

This removes color control from warnings for unstable features.
For some reason this removed color support from `cargo -Zhelp` in the
tests but I can't reproduce it locally.

The most important thing was getting the config fix in.
There are two follow ups
- Can we have the config working *and* color?
- Why did this fail for this field and not the others we already had
  tests for?

I ran out my immediate time box for looking into these.

Fixes rust-lang#13991
@rustbot
Copy link
Collaborator

rustbot commented Jun 1, 2024

r? @ehuss

rustbot has assigned @ehuss.
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

@rustbot
Copy link
Collaborator

rustbot commented Jun 1, 2024

⚠️ Warning ⚠️

  • Pull requests are usually filed against the master branch for this repo, but this one is against rust-1.79.0. Please double check that you specified the right target!

@rustbot rustbot added A-configuration Area: cargo config files and env vars S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 1, 2024
@weihanglo
Copy link
Member

Maybe this is needed? #13964

As lint dropping_copy_types will give suggsetion in this
situation.(Changed in rust-lang/rust#125433)
@weihanglo
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Jun 3, 2024

📌 Commit 947466b has been approved by weihanglo

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 3, 2024
@bors
Copy link
Contributor

bors commented Jun 3, 2024

⌛ Testing commit 947466b with merge ffa9cf9...

@bors
Copy link
Contributor

bors commented Jun 3, 2024

☀️ Test successful - checks-actions
Approved by: weihanglo
Pushing ffa9cf9 to rust-1.79.0...

@bors bors merged commit ffa9cf9 into rust-lang:rust-1.79.0 Jun 3, 2024
21 checks passed
@epage epage deleted the config branch June 3, 2024 16:42
bors added a commit to rust-lang-ci/rust that referenced this pull request Jun 3, 2024
…nglo

[beta-1.79] Update cargo

1 commits in 9ca20fadf04af535ec39d1e18c87cd886185d300..ffa9cf99a594e59032757403d4c780b46dc2c43a
2024-05-22 17:55:16 +0000 to 2024-06-03 14:41:30 +0000
- [beta-1.79] fix(config): Ensure `--config net.git-fetch-with-cli=true` is respected (rust-lang/cargo#13997)

r? ghost
@rustbot rustbot added this to the 1.79.0 milestone Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-configuration Area: cargo config files and env vars S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants