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.

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

Dependency decluttering #44

Merged
merged 7 commits into from
Mar 3, 2024
Merged

Dependency decluttering #44

merged 7 commits into from
Mar 3, 2024

Conversation

fabiangeisler
Copy link
Owner

@fabiangeisler fabiangeisler commented Feb 13, 2024

build(py38): Drop support for python 3.8 and pin requirements
style(typehints): Convert type comments to type annotations
docs(README): Rephrase some FAQ answers
style(mypy): Replace Dict and List with dict and list in type hints
style(all): Use f-strings instead of format function

This brings us in line with the supported versions of shotgun_api3.
This is the more modern approach for doing type hints.
@fabiangeisler fabiangeisler added documentation Improvements or additions to documentation enhancement New feature or request dependencies Pull requests that update a dependency file labels Feb 13, 2024
@fabiangeisler fabiangeisler self-assigned this Feb 13, 2024
Freeze and update all current dependencies

BREAKING CHANGE: This results in the drop of Python 3.8 as it is not supported by the shotgun-api3.
Additionally renamed "ruff-lint" to "ruff-check" in the invoke tasks to be more inline with ruffs naming convention
@fabiangeisler fabiangeisler merged commit 73a0f91 into main Mar 3, 2024
4 checks passed
@fabiangeisler fabiangeisler deleted the dependencie-decluttering branch March 3, 2024 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant