Skip to content

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
@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