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.

Sign up for GitHub

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

Jump to bottom

[cleanup] Misc (for 2024.01 release) #8968

Closed
wants to merge 62 commits into from
Closed

[cleanup] Misc (for 2024.01 release) #8968

wants to merge 62 commits into from

Conversation

bashonly
Copy link
Member

@bashonly bashonly commented Jan 10, 2024 •
edited by Grub4K

Closes #8816

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:

  • I am the original author of this code and I am willing to release it under Unlicense
  • I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

@bashonly
[ie/twitch] Cleanup
e8715c2 
Authored by: bashonly
@bashonly bashonly added the docs/meta/cleanup related to docs, code cleanup, templates, devscripts etc label Jan 10, 2024
@bashonly bashonly mentioned this pull request Jan 10, 2024
Merged
9 tasks
github-advanced-security[bot]
github-advanced-security bot found potential problems Jan 11, 2024
View reviewed changes
yt_dlp/extractor/prosiebensat1.py Fixed Show fixed Hide fixed
yt_dlp/extractor/prosiebensat1.py Fixed Show fixed Hide fixed
@pukkandan pukkandan mentioned this pull request Jan 12, 2024
Merged
@Grub4K Grub4K mentioned this pull request Jan 24, 2024
Closed
10 tasks
@Grub4K Grub4K mentioned this pull request Jan 27, 2024
Merged
9 tasks
bashonly and others added 4 commits January 28, 2024 15:49
@seproDev seproDev mentioned this pull request Feb 2, 2024
Merged
9 tasks
Grub4K
Grub4K requested changes Mar 9, 2024
View reviewed changes
devscripts/__init__.py Outdated Show resolved Hide resolved
pukkandan added a commit to pukkandan/yt-dlp-dev that referenced this pull request Mar 10, 2024
@pukkandan
[cleanup] Fix misc bugs (yt-dlp#8968)
93240fc 
Closes yt-dlp#8816

Authored by: bashonly, seproDev, pukkandan, Grub4k
pukkandan pushed a commit to pukkandan/yt-dlp-dev that referenced this pull request Mar 10, 2024
@seproDev @pukkandan
[cleanup, ie] Match both http and https in _VALID_URL (yt-dlp#8968
a687226 
)

Except for Vimeo, since that causes matching collisions.

Authored by: seproDev
pukkandan added a commit to pukkandan/yt-dlp-dev that referenced this pull request Mar 10, 2024
@pukkandan
[cleanup] Remove unused code (yt-dlp#8968)
ed3bb2b 
Authored by: pukkandan, seproDev
pukkandan added a commit to pukkandan/yt-dlp-dev that referenced this pull request Mar 10, 2024
@pukkandan
[cleanup] Misc (yt-dlp#8968)
615a844 
Authored by: pukkandan, bashonly, seproDev
@Grub4K
Copy link
Member

Grub4K commented Mar 10, 2024

Pushed as 47ab66d...615a844

@Grub4K Grub4K closed this Mar 10, 2024
aalsuwaidi pushed a commit to aalsuwaidi/yt-dlp that referenced this pull request Apr 21, 2024
@pukkandan @aalsuwaidi
[cleanup] Fix misc bugs (yt-dlp#8968)
aacc9ba 
Closes yt-dlp#8816

Authored by: bashonly, seproDev, pukkandan, Grub4k
aalsuwaidi pushed a commit to aalsuwaidi/yt-dlp that referenced this pull request Apr 21, 2024
@seproDev @aalsuwaidi
[cleanup, ie] Match both http and https in _VALID_URL (yt-dlp#8968
558c62d 
)

Except for Vimeo, since that causes matching collisions.

Authored by: seproDev
aalsuwaidi pushed a commit to aalsuwaidi/yt-dlp that referenced this pull request Apr 21, 2024
@pukkandan @aalsuwaidi
[cleanup] Remove unused code (yt-dlp#8968)
a8e4aa2 
Authored by: pukkandan, seproDev
aalsuwaidi pushed a commit to aalsuwaidi/yt-dlp that referenced this pull request Apr 21, 2024
@pukkandan @aalsuwaidi
[cleanup] Misc (yt-dlp#8968)
3987b5a 
Authored by: pukkandan, bashonly, seproDev
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Grub4K Grub4K Grub4K approved these changes

Assignees
No one assigned
Labels
docs/meta/cleanup related to docs, code cleanup, templates, devscripts etc
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Argument of type 'NoneType' is not iterable while running on Docker/airflow
4 participants
@bashonly @Grub4K @seproDev @pukkandan