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

chang 1 tries to 1 try #15328

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

Conversation

zhw2101024
Copy link

What does this PR try to resolve?

The "1 tries remaining" message is boring, "1 try remaining" seems better.

How should we test and review this PR?

Run cargo update without network and watch the output messages.

@rustbot
Copy link
Collaborator

rustbot commented Mar 19, 2025

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

@rustbot rustbot added A-networking Area: networking issues, curl, etc. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 19, 2025
Copy link
Contributor

@epage epage left a comment

Choose a reason for hiding this comment

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

Thanks!

Looks like the tests need to be updated.

@weihanglo
Copy link
Member

@zhw2101024 mind rebasing this onto the latest master branch, so we can then merge it?

@zhw2101024
Copy link
Author

@zhw2101024 mind rebasing this onto the latest master branch, so we can then merge it?

Hi, rebasing finished. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-networking Area: networking issues, curl, etc. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants