Skip to content

Conversation

@google-labs-jules
Copy link
Contributor

The --progress flag was available for all commands, but it's only implemented for GET requests. This change restricts the --progress flag to the GET command to avoid confusion.


PR created automatically by Jules for task 10989237560524555892

The --progress flag was previously available for all commands, but it is only implemented for GET requests. This could be confusing for users.

This commit moves the --progress argument from the common arguments to the GET parser, so it is only available for GET requests.

A new test case has been added to verify that the --progress flag is not available for commands other than GET.
@google-labs-jules
Copy link
Contributor Author

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll acknowledge your comments with a 👀 emoji and then get to work. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

I will automatically address your feedback. For any comments you don't want me to act on, just include (aside).



For security, I will only act on instructions from the user who triggered this task for this pull request.

@mindedal mindedal marked this pull request as ready for review September 14, 2025 10:24
@mindedal mindedal merged commit 225f74e into main Sep 14, 2025
4 checks passed
@mindedal mindedal deleted the fix-progress-flag branch September 14, 2025 10:26
mindedal added a commit that referenced this pull request Oct 29, 2025
The --progress flag was available for all commands, but it's only implemented for GET requests. This change restricts the --progress flag to the GET command to avoid confusion.
mindedal added a commit that referenced this pull request Nov 20, 2025
The --progress flag was available for all commands, but it's only implemented for GET requests. This change restricts the --progress flag to the GET command to avoid confusion.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants