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

tool_getparam: make --show-headers the same as --include #13987

Closed
wants to merge 1 commit into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Jun 23, 2024

Simply a name alias that better explains what the option does.

@bagder bagder added cmdline tool feature-window A merge of this requires an open feature window labels Jun 23, 2024
@github-actions github-actions bot added the tests label Jun 23, 2024
@fogti
Copy link

fogti commented Jun 23, 2024

maybe it would be a good idea to include in the option name that it particularly concerns response headers (from the name alone I would've expected that it shows both request headers and response headers)

@bagder
Copy link
Member Author

bagder commented Jun 23, 2024

maybe it would be a good idea to include in the option name that it particularly concerns response headers

Can you think of such a name that does not also make it veeeeery long?

@bagder bagder self-assigned this Jun 30, 2024
bagder added a commit that referenced this pull request Jul 4, 2024
Simply a name alias that better explains what the option does.

Closes #13987
Simply a name alias that better explains what the option does.

Closes #13987
meslubi2021 pushed a commit to meslubi2021/curl that referenced this pull request Jul 19, 2024
Simply a name alias that better explains what the option does.

Closes curl#13987
@bagder bagder closed this in 82c53f8 Aug 3, 2024
@bagder bagder deleted the bagder/show-headers branch August 3, 2024 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cmdline tool feature-window A merge of this requires an open feature window tests
Development

Successfully merging this pull request may close these issues.

2 participants