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

sort install_requires and sync with mypy deps in pre-commit-config #469

Closed
wants to merge 1 commit into from

Conversation

pacrob
Copy link
Member

@pacrob pacrob commented May 27, 2024

Important to have the same deps for mypy checks as install_requires so users don't get mypy errors we missed.

To-Do

  • Clean up commit history

Cute Animal Picture

image

… make sure mypy catches everything, fix the 2 errors that popped up
@pacrob pacrob force-pushed the add-deps-to-pre-commit-mypy branch from 1471794 to 7a0e1d0 Compare May 27, 2024 19:10
@pacrob pacrob closed this Jul 3, 2024
@pacrob pacrob deleted the add-deps-to-pre-commit-mypy branch July 3, 2024 17:24
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