Skip to content

curl: support -w '%{num_retries} #13910

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

curl: support -w '%{num_retries} #13910

wants to merge 1 commit into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Jun 7, 2024

Suggested-by: Jay Guerette
Ref: #13901

@github-actions github-actions bot added the tests label Jun 7, 2024
jay
jay approved these changes Jun 9, 2024
View reviewed changes
Copy link
Member

@jay jay left a comment

Choose a reason for hiding this comment

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

note two retry variables so similar in name num_retries and retry_numretries different in value might be a little confusing to someone modifying the code in the future. retry_numretries -> retry_remaining?

@bagder
curl: support -w '%{num_retries}
999da41 
Suggested-by: Jay Guerette
Ref: #13901
Closes #13910
@bagder bagder force-pushed the bagder/num_retries branch from c14fd46 to 999da41 Compare June 10, 2024 21:16
@bagder bagder closed this in d69ee3e Jun 11, 2024
@bagder bagder deleted the bagder/num_retries branch June 11, 2024 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jay jay jay approved these changes

Assignees
No one assigned
Labels
cmdline tool tests
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bagder @jay