-
Notifications
You must be signed in to change notification settings - Fork 4
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
Update dependencies 2024 #123
Conversation
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.
Here's some guideposts!
screenpy/configuration.py
Outdated
def _parse_pyproject_toml(tool_path: str) -> dict[str, Any]: | ||
"""Parse a pyproject toml file, pulling out relevant parts for ScreenPy. | ||
class PyprojectTomlConfig(PydanticBaseSettingsSource): | ||
"""Load setting configuration from pyproject.toml.""" |
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.
This was the biggest change.
Pydantic moved its settings management stuff into pydantic-settings
, and changed a whole heckin' lot over there.
Some of the changes actually made things a lot easier for us, 'cause now you can just inherit from ScreenPySettings
when you make your own configuration in an extension. Neat!
When attempting to do some codin', i got an error that ScreenPy required a version of Pydantic that is incompatible with the one i was trying to install.
Well let's fix that, i thought! But there was a pretty big thing to fix. This PR updates the dependencies in pre-commit and poetry, and also updates the copyright dates. Happy 2024!