Skip to content

Conversation

ELC
Copy link

@ELC ELC commented Feb 28, 2025

In this PR the Pipfile is updated with the latest versions of the dependencies, pinning each package.

Python version in the Pipfile was updated to 3.11.

The library youtube-dl was swapped with yt-dlp since the latter has the latest changes to work with Youtube's API

The library sh was replaced with gitpython, this make the script compatible with Windows without any additional changes.

The key license was removed as it was not properly fetched from the Youtube API.

I'm also open to refactor this utility and add tests if there is intention to keep it maintained.

This version was used to create a passing PR in the data repo: pyvideo/data#1212

Please let me know your thoughts @jonafato / @willkg / @logston

ELC added 5 commits February 28, 2025 20:40
- Set specific versions for packages in Pipfile to ensure
  consistency across environments.
- Update Python version requirement from 3.5 to 3.11.
- Prevent tracking of environment variable files.
- Helps maintain security and avoid accidental commits of sensitive data.
- Update `youtube-dl` to `yt-dlp` version 2025.2.19
- Add `gitpython` version 3.1.44
- Ensure compatibility with Python versions
- Use GitPython to handle git operations instead of shell commands.
- Update branch creation and commit methods to utilize the Repo class.
- Change checkout from 'master' to 'main' to align with naming conventions.
- Upgrade pyaml to version 25.5.0
- Upgrade yt-dlp to version 2025.05.22
- Upgrade mypy to version 1.16.0
- Upgrade pudb to version 2025.1
- Update packaging and platformdirs versions
- Add scrape script entry point
@ELC
Copy link
Author

ELC commented Jun 7, 2025

I've just tested this, this could help other people trying to contribute

Do you think we can merge this?

@jonafato / @amyreese

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant