-
-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
[cleanup] Misc (for 2024.04 release) #9426
Conversation
Authored by: bashonly
Authored by: bashonly
Authored by: bashonly
Authored by: bashonly
Authored by: bashonly
Authored by: bashonly
{ | ||
"action": "change", | ||
"when": "e3a3ed8a981d9395c4859b6ef56cd02bc3148db2", | ||
"short": "[cleanup:ie] No `from` stdlib imports in extractors", |
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.
Pretty sure cleanup, ie
is supposed to work 🤔
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.
no, since they are different depths. Full form is misc/cleanup
and extractor
and different depths cannot be merged. Ive asked about this before and since no conflict resolving strategy was proposed I did not implement any
Authored by: bashonly, pukkandan
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?