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
[utils] traverse_obj
: Move is_user_input
into output template
#8673
Conversation
We should try and do It might make sense to instead have that be done when we know what we need for outtmpl, but it would be nice if we could separate concerns and not have a single, massive traversal + outtmpl PR |
We'll be able to split it up after we have the full picture. |
Raise a deprecation_warning if is_user_input is being used. We can remove the param completely after a while. |
I don't think that should be needed. We are explicitly stating that it is private API only. |
Even so, no harm in leaving it for a bit just in case. |
…-dlp#8673) Authored by: Grub4K
* remove `is_user_input` option per yt-dlp/yt-dlp#8673 * support traversal of compat_xml_etree_ElementTree_Element per yt-dlp/yt-dlp#8911 * allow un/branching using all and any per yt-dlp/yt-dlp#9571 * support traversal of compat_cookies.Morsel and multiple types in `set()` keys per yt-dlp/yt-dlp#9577 thx Grub4k for these * also, move traversal tests to a separate class
* remove `is_user_input` option per yt-dlp/yt-dlp#8673 * support traversal of compat_xml_etree_ElementTree_Element per yt-dlp/yt-dlp#8911 * allow un/branching using all and any per yt-dlp/yt-dlp#9571 * support traversal of compat_cookies.Morsel and multiple types in `set()` keys per yt-dlp/yt-dlp#9577 thx Grub4k for these * also, move traversal tests to a separate class
* remove `is_user_input` option per yt-dlp/yt-dlp#8673 * support traversal of compat_xml_etree_ElementTree_Element per yt-dlp/yt-dlp#8911 * allow un/branching using all and any per yt-dlp/yt-dlp#9571 * support traversal of compat_cookies.Morsel and multiple types in `set()` keys per yt-dlp/yt-dlp#9577 thx Grub4k for these * also, move traversal tests to a separate class
* remove `is_user_input` option per yt-dlp/yt-dlp#8673 * support traversal of compat_xml_etree_ElementTree_Element per yt-dlp/yt-dlp#8911 * allow un/branching using all and any per yt-dlp/yt-dlp#9571 * support traversal of compat_cookies.Morsel and multiple types in `set()` keys per yt-dlp/yt-dlp#9577 thx Grub4k for these * also, move traversal tests to a separate class
IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information
Preparations for
traverse_obj
rewrite in combination with output template adjustments.Both private parameters should be moved out of the function and into
outtmpl
instead.Template
Before submitting a pull request make sure you have:
In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:
What is the purpose of your pull request?