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
Deprecated response_format in vso search #7468
Conversation
bab74cc
to
999f746
Compare
|
||
@deprecated_renamed_argument("vso_response_format", None,"6.0", warning_type=SunpyDeprecationWarning) | ||
@deprecated_renamed_argument("progress", None,"6.0", warning_type=SunpyDeprecationWarning) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The other choice is to pass progress down so it works?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why would we deprecate it in full_query
but not here? Maybe I'm missing some context
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I mean more, I could work out how where progress should be passed and then not deprecate it across everything.
But since we never used it, I feel like removing it anyway.
Co-authored-by: David Stansby <dstansby@gmail.com>
2dff6a7
to
9459f23
Compare
Co-authored-by: Will Barnes <will.t.barnes@gmail.com>
Co-authored-by: Will Barnes <will.t.barnes@gmail.com>
Co-authored-by: Will Barnes <will.t.barnes@gmail.com>
Co-authored-by: Alasdair Wilson <60351846+alasdairwilson@users.noreply.github.com>
Thanks for the reviews @wtbarnes and @alasdairwilson |
Supersedes: #7364
So during the 2.1 days we added this old response type to prevent some breaking changes from the rework of the metadata and we never got around to removing it.
So I have finally marked it as deprecated alongside some keywords that were unused.
This should not be backported.