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

Warning when precise or aggressive without -p flag #10988

Merged
merged 2 commits into from
Aug 17, 2022

Conversation

Rustin170506
Copy link
Member

What does this PR try to resolve?

ref #10919.

Warning when precise or aggressive without -p flag. It will be a hard error in future.

How should we test and review this PR?

  • Unit test.

@rust-highfive
Copy link

r? @ehuss

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 15, 2022
@Rustin170506
Copy link
Member Author

Thanks for your review! 💚 💙 💜 💛 ❤️

@Rustin170506 Rustin170506 force-pushed the rustin-patch-warn branch 3 times, most recently from 19816b5 to b6de8b4 Compare August 17, 2022 13:07
Signed-off-by: hi-rustin <rustin.liu@gmail.com>
Signed-off-by: hi-rustin <rustin.liu@gmail.com>
@epage
Copy link
Contributor

epage commented Aug 17, 2022

@bors r+

@bors
Copy link
Contributor

bors commented Aug 17, 2022

📌 Commit a58489d has been approved by epage

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 Aug 17, 2022
@bors
Copy link
Contributor

bors commented Aug 17, 2022

⌛ Testing commit a58489d with merge 9e1b457...

@bors
Copy link
Contributor

bors commented Aug 17, 2022

☀️ Test successful - checks-actions
Approved by: epage
Pushing 9e1b457 to master...

@bors bors merged commit 9e1b457 into rust-lang:master Aug 17, 2022
@Rustin170506 Rustin170506 deleted the rustin-patch-warn branch August 18, 2022 01:00
bors added a commit that referenced this pull request Aug 22, 2022
Add more tests for aggressive or precise update

### What does this PR try to resolve?

When I was working on #10988, I found there is no test for the aggressive update. So I added it.

Also, I added a test for precise update to make sure it won't force update the deps of SPEC.
### How should we test and review this PR?

Unit tests.
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 24, 2022
Update cargo

7 commits in 9809f8ff33c2b998919fd0432c626f0f7323697a..6da726708a4406f31f996d813790818dce837161
2022-08-16 22:10:06 +0000 to 2022-08-23 21:39:56 +0000
- Update non-ASCII crate name warning message (rust-lang/cargo#11017)
- Add more tests for aggressive or precise update (rust-lang/cargo#11011)
- Ignore broken but excluded file during traversing (rust-lang/cargo#11008)
- Improve error message for wrong target names (rust-lang/cargo#10999)
- Bump snapbox to 0.3 (rust-lang/cargo#11005)
- remove missed reference to workspace inheritance in unstable.md (rust-lang/cargo#11001)
- Warning when precise or aggressive without -p flag (rust-lang/cargo#10988)
@ehuss ehuss added this to the 1.65.0 milestone Aug 31, 2022
bors added a commit that referenced this pull request Nov 30, 2022
Error when precise without -p flag

### What does this PR try to resolve?

close #10919

Follow up #10988, see #10919 (comment)

### How should we test and review this PR?

You can manually build and test it. You can try `cargo update --precise xxx` without -p flag.

### Additional information
It has already been released on stable. `rustc 1.65.0 (897e37553 2022-11-02)`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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