Skip to content

Loosen version restrictions #3

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jpenney
Copy link

@jpenney jpenney commented Oct 26, 2024

  • lower minimum Python supported to 3.8
  • if libass is not found by find_library, will try ASS_RENDERER_LIBASS_PATH environment variable
  • removed poetry.lock
  • added mypy to pre-commit
    • fixed some type errors
  • added pylint to pre-commit
    • disabled docstring checks for now

Verified all tests and pre-commit hooks pass

Tested on various Python versions on Windows, Ubuntu, macOS

    - lower minimum Python supported to 3.8
    - if libass is not found by find_library, will try ASS_RENDERER_LIBASS_PATH environment variable
    - removed poetry.lock
        - prevented installation of library in existing envs without serious corruption
        - see https://python-poetry.org/docs/basic-usage/#as-a-library-developer
    - added mypy to pre-commit
        - fixed some type errors
    - added pylint to pre-commit
        - disabled docstring checks for now

Verified all tests and pre-commit hooks pass
@jpenney jpenney changed the title Loosen version restrictions to allow installation in existing envs Loosen version restrictions Oct 26, 2024
@jpenney
Copy link
Author

jpenney commented Oct 26, 2024

See also: bubblesub/ass_parser#1, #3

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