search

chroma-core / chroma

the AI-native open-source embedding database
https://www.trychroma.com/
Apache License 2.0
13.5k stars 1.15k forks source link

Enable mypy lint in CI #2054

Open codetheweb opened 2 months ago

codetheweb commented 2 months ago

I recently enabled most of our lints in CI: https://github.com/chroma-core/chroma/pull/2046.

I didn't enable mypy in that PR because it looked like it required a few manual changes to pass the lint. We need to enable this lint by updating .github/workflows/chroma-lint.yml and making the appropriate changes to pass the lint (test locally with pre-commit run --all-files mypy).

ibratoev commented 2 months ago

@codetheweb You can assign this to me. There are a ton of errors to fix. I will start opening PRs in batches to make it easier to review.

codetheweb commented 2 months ago

awesome, thank you so much!